18 Commits

Author SHA1 Message Date
Lipu Fei
f867f3485e Fix TestIntentManager 2019-10-01 11:30:54 +02:00
Ghostkeeper
4e8929a072
Fix structure of mock container tree
Found during testing and tooling.
2019-09-27 15:26:14 +02:00
Ghostkeeper
97eaeab3bd
Fix missing intent in mock and missing quality_type
These are used by the code. Otherwise there would be an AttributeError of 'intent' not existing, and the quality type in the quality node wouldn't match because it's a magic mock.
2019-09-27 15:13:23 +02:00
Jaime van Kessel
4dc918c647
Fix unit test 2019-09-23 17:09:54 +02:00
Ghostkeeper
557746a832
Partial fix for intent manager not finding correct profile in test
Only in test.
2019-09-23 15:17:15 +02:00
Ghostkeeper
7d40d962e8
Mock ContainerTree instead of ContainerRegistry to test intent manager
Because now it relies on the container tree instead of querying the registry anew every time.
2019-09-23 15:03:24 +02:00
Ghostkeeper
be36ae278b
Fix tests mocking container tree instead of quality manager
Contributes to issue CURA-6600.
2019-08-26 16:23:20 +02:00
Jaime van Kessel
6cea609b56 Fix intentManager test
Since we changed how the QualityManager is constructed, the test should change as well

CURA-6600
2019-08-12 14:52:05 +02:00
Lipu Fei
8863516aa4 Rename function name and fix tests
CURA-6597
2019-07-09 12:53:44 +02:00
Ghostkeeper
168c9db648
Code style: Space around binary operators
Contributes to issue CURA-6597.
2019-07-04 09:28:23 +02:00
Remco Burema
50cbf71f68 Fixed test 'selectIntent' for IntentManager-tests.
part of CURA-6091
2019-07-02 13:15:18 +02:00
Remco Burema
ffe9515236 Refactor: Split up single test method in smaller ones.
part of CURA-6091
2019-07-02 08:57:40 +02:00
Remco Burema
810fee37eb Start to test other IntentManager functions.
Very rudimentary at the moment, need to split the method into 3, and make a class for the setup.
This also uncovered that the currentAvailableIntents doesn't (unless the global stack is missing) retrun any default intents, while currentAvailableIntentCategories does do that.
Since it's not clear how we're going to handle that right now, I made a TODO in the code, which of course will have to be fixed before this/these branch/es are merged.

part of CURA-6091
2019-06-28 12:10:39 +02:00
Remco Burema
2843fc903b First OK test-case for 'currentAvailableIntents'.
part of CURA-6091
2019-06-28 11:09:42 +02:00
Remco Burema
e8a1c68d92 Added (partial) test for 'currentAvailableIntents'.
part of CURA-6091
2019-06-27 17:07:26 +02:00
Remco Burema
02516f0f47 Refactor: Move common fixtures to 'conftest'.
Part of CURA-6091.
2019-06-21 14:58:55 +02:00
Remco Burema
0f9de9935e Add unit-test for .intentCategories
Part of CURA-6091.
2019-06-21 14:37:10 +02:00
Remco Burema
53c387f34d Add TestIntentManager (mostly a dummy file), prevents future conflicts. [CURA-6091] 2019-06-20 16:04:37 +02:00