Skip to content

Reintroduce unit tests#214

Merged
DanielElisenberg merged 10 commits intoworkflow_multitenancy_testsfrom
reintroduce_unit_tests
Dec 1, 2025
Merged

Reintroduce unit tests#214
DanielElisenberg merged 10 commits intoworkflow_multitenancy_testsfrom
reintroduce_unit_tests

Conversation

@DanielElisenberg
Copy link
Collaborator

Reintroduce unit tests

These are the same unit tests on main, minus the tests for the models that are now moved to the local storage. Another PR will be posted for these.

Some very minor changes:

  • LocalStorageAdapter tests has their own resource dir
  • Some renaming
  • Updated manager tests

@DanielElisenberg DanielElisenberg requested a review from a team as a code owner November 28, 2025 13:33
Copy link
Contributor

@pawbu pawbu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍 Agree with SonarQube warnings

Copy link
Contributor

@haraldsvik haraldsvik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

Its abit hard to see whats changed on github but it works quite nice when looking at it locally and can compare the branch vs main



@dataclass
class MockedWorker:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 1, 2025

@DanielElisenberg DanielElisenberg merged commit 70e2590 into workflow_multitenancy_tests Dec 1, 2025
2 checks passed
@DanielElisenberg DanielElisenberg deleted the reintroduce_unit_tests branch December 1, 2025 14:35
DanielElisenberg added a commit that referenced this pull request Jan 14, 2026
* workflow_multitenancy_tests: move code from app

* workflow_multitenancy_tests: move code to manager

* workflow_multitenancy_tests: update to use job context

* workflow_multitenancy_tests: restart from scratch tests

* workflow_multitenancy_tests: gitkeeps

* Remove dotenv (#212)

* remove_dotenv: move to conftest

* remove_dotenv: remove dependency

* More integration tests (#213)

* more_integration_tests: new datastore resouce dirs

* more_integration_tests: add to common functions

* more_integration_tests: add rollback tests

* more_integration_tests: fix

* more_integration_tests: fix

* Reintroduce unit tests (#214)

* reintroduce_unit_tests: adapter tests

* reintroduce_unit_tests: config tests

* reintroduce_unit_tests: manager tests

* reintroduce_unit_tests: worker tests

* reintroduce_unit_tests: fix resources

* reintroduce_unit_tests: fixes manager test

* reintroduce_unit_tests: fixes

* reintroduce_unit_tests: missing gitkeep

* reintroduce_unit_tests: fix imports

* reintroduce_unit_tests: test fixes

* update_urllib3: update (#215)

* reintroduce_fs_models_test: reintroduce (#217)

* Check tmp dir of all datastores (#219)

* pin_actions: better security (#210)

* Use variables from env in workflow scripts

* remove double env in workflow-step

* update INTEGRATION_TEST_WORKFLOW and DEPLOY_WORKFLOW in env

* Update urllib3

* Check tmp dir of all datastores

---------

Co-authored-by: linnbjornerud <linnchb@gmail.com>
Co-authored-by: Linn Bjørnerud <125447659+linnbjornerud@users.noreply.github.com>

* Extra assertions integration tests (#221)

* extra_assertions_integration_tests: added metadata update assertions

* extra_assertions_integration_tests: more assertions, update resources

* extra_assertions_integration_tests: update dependencies

* extra_assertions_integration_tests: ruff

---------

Co-authored-by: pawbu <pawbu@users.noreply.github.com>
Co-authored-by: linnbjornerud <linnchb@gmail.com>
Co-authored-by: Linn Bjørnerud <125447659+linnbjornerud@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants