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

Add mixin class to replace mandatory hooks #177

Merged
merged 53 commits into from
Oct 9, 2024

Commits on Aug 19, 2024

  1. Merge branch 'main' of github.com:EESSI/test-suite

    Caspar van Leeuwen committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    71e2ea5 View commit details
    Browse the repository at this point in the history

Commits on Sep 4, 2024

  1. Merge branch 'main' of github.com:EESSI/test-suite

    Caspar van Leeuwen committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    fbfe08d View commit details
    Browse the repository at this point in the history
  2. First attempt at using a mixin class

    Caspar van Leeuwen committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    3ae63d8 View commit details
    Browse the repository at this point in the history
  3. Expand the number of hooks in the eessi_mixin class. Check that varia…

    …bles that are needed for those hooks are defined in time, and if not, print a clear message
    Caspar van Leeuwen committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    56d457b View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    24a0256 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    2bdd84f View commit details
    Browse the repository at this point in the history
  6. Expand comment

    Caspar van Leeuwen committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    3ad1662 View commit details
    Browse the repository at this point in the history
  7. Add required_mem_per_node comes from self

    Caspar van Leeuwen committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    119104e View commit details
    Browse the repository at this point in the history
  8. Make sure the required_mem_per_node function returns the value

    Caspar van Leeuwen committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    67d3e4e View commit details
    Browse the repository at this point in the history
  9. More clear SyntaxError message

    Caspar van Leeuwen committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    f096cb9 View commit details
    Browse the repository at this point in the history
  10. Added some more suggestions for the future

    Caspar van Leeuwen committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    bee51d2 View commit details
    Browse the repository at this point in the history

Commits on Sep 19, 2024

  1. Clarify printed message

    Caspar van Leeuwen committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    3ffe7eb View commit details
    Browse the repository at this point in the history
  2. Add memory hook. Make it useable by setting measure_memory_usage

    Caspar van Leeuwen committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    c4a0bcc View commit details
    Browse the repository at this point in the history
  3. Leave a breadcrumb for our future selves

    Caspar van Leeuwen committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    2f84687 View commit details
    Browse the repository at this point in the history
  4. Do dynamic version generation, both at build time and runtime. Use th…

    …e dynamic version determined at runtime and store it as a variable in the eessi_mixin class, so it ends up in all tests that are run. This way, you can alwyas see from the run report of reframe which EESSI test-suite version was used
    Caspar van Leeuwen committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    582b7b6 View commit details
    Browse the repository at this point in the history
  5. First try to get version from installed package. If packate is not in…

    …stalled, try to get it from the git tree
    Caspar van Leeuwen committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    97e60ab View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2024

  1. Make sure scm writes a version file. If the version file doesn't exis…

    …t, we're probably in a git-cloned repo, and we resort to using setuptools_scm at runtime to determine the version
    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    bf48c2f View commit details
    Browse the repository at this point in the history
  2. Of course, the extension shouldn't be part of the import...

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    5095c4f View commit details
    Browse the repository at this point in the history
  3. Import was failing, try something else

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    3c2f0c6 View commit details
    Browse the repository at this point in the history
  4. Now correctly import __version__ in the global namespace of eessi/tes…

    …tsuite/__init__.py
    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    04ca0ca View commit details
    Browse the repository at this point in the history
  5. Fix some flake8 errors

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    28376b7 View commit details
    Browse the repository at this point in the history
  6. Fix some more flake8 issues

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    439c425 View commit details
    Browse the repository at this point in the history
  7. Fix more flake8 issues

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    72d9a39 View commit details
    Browse the repository at this point in the history
  8. Fix more flake8 stuff

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    7909b2d View commit details
    Browse the repository at this point in the history
  9. Remove whitespace on blank line

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    204a819 View commit details
    Browse the repository at this point in the history
  10. Enforce new enough setuptools_scm so that it knows the version_file a…

    …rgument
    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    a396f6a View commit details
    Browse the repository at this point in the history
  11. Make usage of write_to or version_file dependent on the version of py…

    …thon (and thus the version of setuptools_scm) used
    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    c821106 View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    536a444 View commit details
    Browse the repository at this point in the history
  13. Make things conditional on version of setuptools_scm directly

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    84178cd View commit details
    Browse the repository at this point in the history
  14. No longer enforce version v8 or higher for setuptools_scm, it doesn't…

    … work for older python
    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    d90e831 View commit details
    Browse the repository at this point in the history
  15. Remove stray line

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    95a6d6b View commit details
    Browse the repository at this point in the history
  16. Make flake8 happy again

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    a79b263 View commit details
    Browse the repository at this point in the history
  17. Just stick to write_to for everything, and keep the setuptools_scm ve…

    …rsion below 8 to stay compatible with python 3.6. Once we don't care about 3.6 compatibility anymore, we can upgrade to setuptools_scm and use version_file instead of write_to
    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    3f0d174 View commit details
    Browse the repository at this point in the history
  18. Remove import that isn't used

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    430dc10 View commit details
    Browse the repository at this point in the history
  19. Fix CI issue for python 3.6

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    8bbb582 View commit details
    Browse the repository at this point in the history
  20. Fix version of setuptools_scm below 8

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    25f0167 View commit details
    Browse the repository at this point in the history
  21. Stick to setuptools_scm < 7 to be compatible with python 3.6

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    618df8e View commit details
    Browse the repository at this point in the history
  22. Make it < 7 consistently...

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    e0295ea View commit details
    Browse the repository at this point in the history
  23. Configuration menu
    Copy the full SHA
    070a549 View commit details
    Browse the repository at this point in the history
  24. Make things consistent..

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    5108a85 View commit details
    Browse the repository at this point in the history
  25. let setup.py handle the setuptools_scm conditionally, see if that wor…

    …ks...
    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    c8bf4a6 View commit details
    Browse the repository at this point in the history
  26. remove stray line

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    d531e41 View commit details
    Browse the repository at this point in the history
  27. Fix flake8

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    645bb18 View commit details
    Browse the repository at this point in the history
  28. Make sure packaging version isn't too new for Python 3.6

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    ef985d1 View commit details
    Browse the repository at this point in the history
  29. Pass list of strings

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    a6ea23c View commit details
    Browse the repository at this point in the history
  30. Now really fix it

    Caspar van Leeuwen committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    1f1ee2d View commit details
    Browse the repository at this point in the history

Commits on Sep 21, 2024

  1. Merge branch 'use_mixin_class' of github.com:casparvl/test-suite into…

    … use_mixin_class
    Caspar van Leeuwen committed Sep 21, 2024
    Configuration menu
    Copy the full SHA
    446e4c8 View commit details
    Browse the repository at this point in the history
  2. Split off the setuptools_scm versioning into a different pr EESSI#185

    …. The current PR now depends on this, as it relies on the eessi/testsuite/__init__.py file to provide a version
    Caspar van Leeuwen committed Sep 21, 2024
    Configuration menu
    Copy the full SHA
    38ffee1 View commit details
    Browse the repository at this point in the history

Commits on Oct 6, 2024

  1. update mixin class

    Samuel Moors committed Oct 6, 2024
    Configuration menu
    Copy the full SHA
    94d16e7 View commit details
    Browse the repository at this point in the history

Commits on Oct 7, 2024

  1. do not override time_limit

    Samuel Moors committed Oct 7, 2024
    Configuration menu
    Copy the full SHA
    b69d1ee View commit details
    Browse the repository at this point in the history

Commits on Oct 8, 2024

  1. Merge pull request #2 from smoors/pr177

    update mixin class
    casparvl authored Oct 8, 2024
    Configuration menu
    Copy the full SHA
    2f6b33a View commit details
    Browse the repository at this point in the history

Commits on Oct 9, 2024

  1. Merge branch 'use_mixin_class' of github.com:casparvl/test-suite into…

    … use_mixin_class
    Caspar van Leeuwen committed Oct 9, 2024
    Configuration menu
    Copy the full SHA
    7e66d57 View commit details
    Browse the repository at this point in the history
  2. Some cleanup

    Caspar van Leeuwen committed Oct 9, 2024
    Configuration menu
    Copy the full SHA
    24951c8 View commit details
    Browse the repository at this point in the history