Skip to content

Commit a84ecb3

Browse files
Strum355claude
andcommitted
chore(test): remove GITHUB_ACTIONS timeout ternaries
Use the larger timeout value unconditionally instead of branching on the CI environment variable. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 62f12c0 commit a84ecb3

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

test/providers/javascript.test.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ suite('testing the javascript-npm data provider', async () => {
9090

9191
compareSboms(providedDataForStack.content, expectedSbom);
9292

93-
}).timeout(process.env.GITHUB_ACTIONS ? 30000 : 10000);
93+
}).timeout(30000);
9494
test(`verify package.json data provided for ${providerName} - component analysis - ${scenario}`, async () => {
9595
// load the expected list for the scenario
9696
let expectedSbom = fs.readFileSync(`test/providers/tst_manifests/js-common/${testCase}/component_expected_sbom.json`,).toString().trim()
@@ -102,7 +102,7 @@ suite('testing the javascript-npm data provider', async () => {
102102
let providedDataForComponent = provider.provideComponent(manifestPath);
103103

104104
compareSboms(providedDataForComponent.content, expectedSbom);
105-
}).timeout(process.env.GITHUB_ACTIONS ? 15000 : 10000)
105+
}).timeout(15000)
106106

107107
});
108108

@@ -122,7 +122,7 @@ suite('testing the javascript-npm data provider', async () => {
122122

123123
// Then the SBOM should contain the member's transitive dependencies
124124
compareSboms(result.content, expectedSbom);
125-
}).timeout(process.env.GITHUB_ACTIONS ? 30000 : 10000);
125+
}).timeout(30000);
126126

127127
/// Verifies that component analysis resolves direct dependencies for a workspace member.
128128
test(`verify workspace member data provided for ${providerName} - component analysis`, async () => {
@@ -137,7 +137,7 @@ suite('testing the javascript-npm data provider', async () => {
137137

138138
// Then the SBOM should contain only the member's direct dependencies
139139
compareSboms(result.content, expectedSbom);
140-
}).timeout(process.env.GITHUB_ACTIONS ? 15000 : 10000);
140+
}).timeout(15000);
141141
});
142142

143143
test('loads a valid manifest with ignored dependencies', () => {

0 commit comments

Comments
 (0)