[riot-notifications] [RIOT-OS/RIOT] murdock: enable esp32-wroom-32 for CI testing (#11449)

Kaspar Schleiser notifications at github.com
Mon Apr 29 15:40:11 CEST 2019


kaspar030 commented on this pull request.



> @@ -22,9 +22,9 @@
 )
 
 EXPECTED_PS = (
-    '\tpid | state    Q | pri',
-    '\t  1 | pending  Q |  15',
-    '\t  2 | running  Q |   7'
+    r'\s+pid | state    Q | pri',
+    r'\s+1 | pending  Q |\s+\d+',

> What couple of tests fail? I would like to take a look why.

Take a look at the CI output: https://ci.riot-os.org/RIOT-OS/RIOT/11449/3c5e7c9b76d0f2da5629c9298ae3234b8f75c2c1/output.html

I only skimmed the outputs. Some are expecting default priorities, some fail for other reasons.

> IMHO, it is a more general problem. If there is a variable that allows to define the number of priorities, I would like to be able to use it. Not using it because some tests fail doesn't seem to be the right solution.

Sure, but rearranging one architecture's configuration a bit (to make it more like all the others) is IMO preferable than making the tests less exact (by accepting *any* priority instead of what it should be in the default configuration.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11449#discussion_r279367506
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190429/bb9ea49f/attachment.html>


More information about the notifications mailing list