Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

try to avoid recording tests #354

Merged
merged 6 commits into from
Jul 26, 2024
Merged

try to avoid recording tests #354

merged 6 commits into from
Jul 26, 2024

Commits on Jul 19, 2024

  1. fix: show config packages on startup

    In addition to the path of config file, show the packages property, too.
    apotterri committed Jul 19, 2024
    Configuration menu
    Copy the full SHA
    feec761 View commit details
    Browse the repository at this point in the history
  2. fix: add APPMAP_INSTRUMENT_PROPERTIES

    Add APPMAP_INSTRUMENT_PROPERTIES to control whether properties should be
    instrumented.
    apotterri committed Jul 19, 2024
    Configuration menu
    Copy the full SHA
    11b6307 View commit details
    Browse the repository at this point in the history

Commits on Jul 26, 2024

  1. fix: improve property handling

    Base the decision to instrument a property on the property name, rather
    than trying to use the name of the f{get,set,del} functions. This aligns
    them with the rest of the class's members, e.g. for exclusion.
    
    Also, ensure that those functions only get instrumented once.
    apotterri committed Jul 26, 2024
    Configuration menu
    Copy the full SHA
    5cce0f0 View commit details
    Browse the repository at this point in the history
  2. fix: try to avoid recording tests

    When creating the default config, ignore directories with names that
    match the regex .*test.*. This should avoid instrumenting the majority
    of test functions. For those that do still get instrumented, have the
    test framework integration disable the wrapt wrapped on it, thereby
    disabling recording.
    apotterri committed Jul 26, 2024
    Configuration menu
    Copy the full SHA
    1847b0e View commit details
    Browse the repository at this point in the history
  3. fix: add ruff

    Add ruff to dev dependencies, along with an example config. At some
    point in the future, we may want to switch to using it, rather than
    pylint.
    apotterri committed Jul 26, 2024
    Configuration menu
    Copy the full SHA
    ac94204 View commit details
    Browse the repository at this point in the history
  4. test: pin incremental

    The latest update to incremental (which twisted depends on), appears to
    be broken. Pin to the previous version.
    apotterri committed Jul 26, 2024
    Configuration menu
    Copy the full SHA
    7538fa7 View commit details
    Browse the repository at this point in the history