feat: ignore conditional imports

Author: dominykas

lib/travis/index.js

@@ -25,25 +25,27 @@ internals.toArray = (v) => {
 
 internals.normalizeImports = (travisYaml, { relativeTo }) => {
 
-    return internals.toArray(travisYaml.import).map((entry) => {
+    return internals.toArray(travisYaml.import)
+        .map((entry) => {
 
-        if (typeof entry === 'string') {
-            entry = { source: entry };
-        }
+            if (typeof entry === 'string') {
+                entry = { source: entry };
+            }
 
-        if (entry.source.startsWith('./')) {
-            entry.source = entry.source.substring(2);
+            if (entry.source.startsWith('./')) {
+                entry.source = entry.source.substring(2);
 
-            if (relativeTo) {
-                const relativeParts = relativeTo.source.split('/');
-                relativeParts.pop();
-                relativeParts.push(entry.source);
-                entry.source = relativeParts.join('/');
+                if (relativeTo) {
+                    const relativeParts = relativeTo.source.split('/');
+                    relativeParts.pop();
+                    relativeParts.push(entry.source);
+                    entry.source = relativeParts.join('/');
+                }
             }
-        }
 
-        return entry;
-    });
+            return entry;
+        })
+        .filter((entry) => !entry.if); // @todo: log a warning
 };
 
 

test/fixtures/travis-ymls/testing-imports/conditional.yml

@@ -0,0 +1,4 @@
+language: node_js
+import:
+  - source: partials/indirect-node-14.yml
+    if: branch = master

test/travis.js

@@ -103,4 +103,26 @@ describe('.travis.yml parsing', () => {
         });
     });
 
+    it('ignores conditional imports', async () => {
+
+        await fixture.setupRepoFolder({
+            partials: true,
+            travisYml: `testing-imports/conditional.yml`
+        });
+
+        const result = await NodeSupport.detect({ path: fixture.path });
+
+        internals.assertCommit(result);
+
+        expect(result).to.equal({
+            name: 'test-module',
+            version: '0.0.0-development',
+            timestamp: 1580673602000,
+            travis: {
+                raw: ['latest'],
+                resolved: { 'latest': '13.14.0' }
+            }
+        });
+    });
+
 });