test/test_targets: skip Minerva test on Travis-CI, remove commented tests

diff --git a/test/test_targets.py b/test/test_targets.py
index 038faedb2bfe676a583e18c9b41e8b5b86d49c40..fc5604ac1c1a5ec735d13283945f0bdf717d76cf 100644 (file)
--- a/test/test_targets.py
+++ b/test/test_targets.py
@@ -143,23 +143,23 @@ litex/boards/targets/simple.py litex.boards.platforms.arty \
         subprocess.check_output(cmd, shell=True)
 
     # Build some variants for the arty platform to make sure they work.
-    def test_variants_riscv(self):
+    def test_variants_picorv32(self):
+        self.run_variants("picorv32", ('standard', 'minimal'))
+
+    def test_variants_vexriscv(self):
+        self.run_variants("vexriscv", ('standard', 'minimal', 'lite', 'lite+debug', 'full+debug'))
+
+    @unittest.skipIf(RUNNING_ON_TRAVIS, "No nMigen/Yosys on Travis-CI")
+    def test_variants_minerva(self):
+        self.run_variants("minerva", ('standard',))
+
+    def test_variants_vexriscv(self):
         cpu_variants = {
-            'picorv32': ('standard', 'minimal'),
             'vexriscv': ('standard', 'minimal', 'lite', 'lite+debug', 'full+debug'),
-            'minerva': ('standard',),
         }
         for cpu, variants in cpu_variants.items():
             self.run_variants(cpu, variants)
 
-    #def test_bad_variants(self):
-    #    with self.assertRaises(subprocess.CalledProcessError):
-    #        self.run_variant('vexriscv', 'bad')
-
-    #def test_bad_variant_extension(self):
-    #    with self.assertRaises(subprocess.CalledProcessError):
-    #        self.run_variant('vexriscv', 'standard+bad')
-
     @unittest.skipIf(RUNNING_ON_TRAVIS, "No lm32 toolchain on Travis-CI")
     def test_variants_lm32(self):
         self.run_variants('lm32', ('standard', 'minimal', 'lite'))