Skip to content

Commit 75311d5

Browse files
committed
tests
Signed-off-by: Jan Kowalleck <jan.kowalleck@gmail.com>
1 parent 84d7fa0 commit 75311d5

14 files changed

Lines changed: 45 additions & 64 deletions

HISTORY.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,9 @@ All notable changes to this project will be documented in this file.
99
* BREAKING changes
1010
* Removed deprecated symbols
1111
* Removed
12+
* Entrypoint `Builders` (via [#1377])
13+
* Entrypoint `Factories` (via [#1377])
14+
* Entrypoint `Utils` (via [#1377])
1215
* Deprecated symbol `Builders` ([#1346] via [#1377])
1316
* Deprecated symbol `Builders.FromNodePackageJson` ([#1346] via [#1377])
1417
* Deprecated symbol `Builders.FromNodePackageJson.ToolBuilder` ([#1346] via [#1377])

package.json

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -145,18 +145,10 @@
145145
"default": "./dist.node/index.node.js"
146146
},
147147
"./package.json": "./package.json",
148-
"./Builders": {
149-
"types": "./dist.d/builders/index.node.d.ts",
150-
"default": "./dist.node/builders/index.node.js"
151-
},
152148
"./Enums": {
153149
"types": "./dist.d/enums/index.d.ts",
154150
"default": "./dist.node/enums/index.js"
155151
},
156-
"./Factories": {
157-
"types": "./dist.d/factories/index.node.d.ts",
158-
"default": "./dist.node/factories/index.node.js"
159-
},
160152
"./Models": {
161153
"types": "./dist.d/models/index.d.ts",
162154
"default": "./dist.node/models/index.js"
@@ -177,10 +169,6 @@
177169
"types": "./dist.d/types/index.d.ts",
178170
"default": "./dist.node/types/index.js"
179171
},
180-
"./Utils": {
181-
"types": "./dist.d/utils/index.node.d.ts",
182-
"default": "./dist.node/utils/index.node.js"
183-
},
184172
"./Validation": {
185173
"types": "./dist.d/validation/index.node.d.ts",
186174
"default": "./dist.node/validation/index.node.js"

tests/integration/Builders.FromNodePackageJson.ComponentBuilder.test.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,17 +24,16 @@ const { suite, test } = require('mocha')
2424
const {
2525
Enums,
2626
Models,
27-
Factories,
28-
Builders: { FromNodePackageJson: { ComponentBuilder } }
27+
Contrib,
2928
} = require('../../')
3029

3130
suite('integration: Builders.FromNodePackageJson.ComponentBuilder', () => {
3231
const salt = Math.random()
3332

34-
const extRefFactory = new Factories.FromNodePackageJson.ExternalReferenceFactory()
35-
const licenseFactory = new Factories.LicenseFactory()
33+
const extRefFactory = new Contrib.FromNodePackageJson.Factories.ExternalReferenceFactory()
34+
const licenseFactory = new Contrib.License.Factories.LicenseFactory()
3635

37-
const sut = new ComponentBuilder(extRefFactory, licenseFactory);
36+
const sut = new Contrib.FromNodePackageJson.Builders.ComponentBuilder(extRefFactory, licenseFactory);
3837

3938
[
4039
[

tests/integration/Builders.FromNodePackageJson.ToolBuilders.test.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,16 +24,16 @@ const { suite, test } = require('mocha')
2424
const {
2525
Models,
2626
Factories,
27-
Builders: { FromNodePackageJson: { ToolBuilder } }
27+
Contrib,
2828
} = require('../../')
2929

3030
suite('integration: Builders.FromNodePackageJson.ToolBuilder', () => {
3131
const salt = Math.random()
3232

33-
const extRefFactory = new Factories.FromNodePackageJson.ExternalReferenceFactory()
33+
const extRefFactory = new Contrib.FromNodePackageJson.Factories.ExternalReferenceFactory()
3434
extRefFactory.makeExternalReferences = () => [`FAKE REFERENCES ${salt}`]
3535

36-
const sut = new ToolBuilder(extRefFactory)
36+
const sut = new Contrib.FromNodePackageJson.Builders.ToolBuilder(extRefFactory)
3737

3838
const data = {
3939
name: '@foo/bar',

tests/integration/Factories.FromNodePackageJson.ExternalReferenceFactory.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,11 @@ const { suite, test } = require('mocha')
2424
const {
2525
Enums: { ExternalReferenceType, HashAlgorithm },
2626
Models: { ExternalReference, HashDictionary },
27-
Factories: { FromNodePackageJson: { ExternalReferenceFactory } }
27+
Contrib,
2828
} = require('../../')
2929

3030
suite('integration: Factories.FromNodePackageJson.ExternalReferenceFactory', () => {
31-
const sut = new ExternalReferenceFactory()
31+
const sut = new Contrib.FromNodePackageJson.Factories.ExternalReferenceFactory()
3232

3333
suite('from "homepage"', () => {
3434
test('is non-empty string', () => {

tests/integration/Factories.FromNodePackageJson.PackageUrlFactory.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,13 @@ const { PackageURL } = require('packageurl-js')
2525
const {
2626
Enums,
2727
Models,
28-
Factories: { FromNodePackageJson: { PackageUrlFactory } }
28+
Contrib,
2929
} = require('../../')
3030

3131
suite('integration: Factories.FromNodePackageJson.PackageUrlFactory', () => {
3232
const salt = Math.random()
3333

34-
const sut = new PackageUrlFactory('testing')
34+
const sut = new Contrib.PackageUrl.Factories.PackageUrlFactory('testing')
3535

3636
suite('makeFromComponent', () => {
3737
test('no-name-no-purl', () => {

tests/integration/Factories.LicenseFactory.test.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,13 @@ const assert = require('node:assert')
2222
const { suite, test } = require('mocha')
2323

2424
const {
25-
Factories: { LicenseFactory },
25+
Contrib,
2626
Models: { LicenseExpression, NamedLicense, SpdxLicense }
2727
} = require('../../')
2828

2929
suite('integration: Factories.LicenseFactory', () => {
3030
test('makeFromString() -> LicenseExpression', () => {
31-
const sut = new LicenseFactory()
31+
const sut = new Contrib.License.Factories.LicenseFactory()
3232
const expression = '(MIT OR Apache-2.0)'
3333

3434
const license = sut.makeFromString(expression)
@@ -38,7 +38,7 @@ suite('integration: Factories.LicenseFactory', () => {
3838
})
3939

4040
test('makeFromString() -> NamedLicense', () => {
41-
const sut = new LicenseFactory()
41+
const sut = new Contrib.License.Factories.LicenseFactory()
4242

4343
const license = sut.makeFromString('(c) foo bar')
4444

@@ -49,7 +49,7 @@ suite('integration: Factories.LicenseFactory', () => {
4949
})
5050

5151
test('makeFromString() -> SpdxLicense', () => {
52-
const sut = new LicenseFactory()
52+
const sut = new Contrib.License.Factories.LicenseFactory()
5353

5454
const license = sut.makeFromString('MIT')
5555

tests/integration/Factories.PackageUrlFactory.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,13 @@ const { PackageURL } = require('packageurl-js')
2525
const {
2626
Enums,
2727
Models,
28-
Factories: { PackageUrlFactory }
28+
Contrib,
2929
} = require('../../')
3030

3131
suite('integration: Factories.PackageUrlFactory', () => {
3232
const salt = Math.random()
3333

34-
const sut = new PackageUrlFactory('testing')
34+
const sut = new Contrib.PackageUrl.Factories.PackageUrlFactory('testing')
3535

3636
suite('makeFromComponent', () => {
3737
test('no-name-no-purl', () => {

tests/integration/Utils.LicenseUtility.LicenseEvidenceGatherer.node.test.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,13 @@ const { suite, test } = require('mocha')
2626
const {
2727
Models: { Attachment },
2828
Enums: { AttachmentEncoding },
29-
Utils: { LicenseUtility: { LicenseEvidenceGatherer } }
29+
Contrib,
3030
} = require('../../')
3131

3232
suite('integration: Utils.LicenseUtility.LicenseEvidenceGatherer', () => {
3333
test('no path -> throws', () => {
3434
const { fs } = memfs({ '/': {} })
35-
const leg = new LicenseEvidenceGatherer({ fs })
35+
const leg = new Contrib.License.Utils.LicenseEvidenceGatherer({ fs })
3636
assert.throws(
3737
() => {
3838
Array.from(
@@ -48,7 +48,7 @@ suite('integration: Utils.LicenseUtility.LicenseEvidenceGatherer', () => {
4848

4949
test('no files', () => {
5050
const { fs } = memfs({ '/': {} })
51-
const leg = new LicenseEvidenceGatherer({ fs })
51+
const leg = new Contrib.License.Utils.LicenseEvidenceGatherer({ fs })
5252
const errors = []
5353
const found = Array.from(
5454
leg.getFileAttachments(
@@ -66,7 +66,7 @@ suite('integration: Utils.LicenseUtility.LicenseEvidenceGatherer', () => {
6666
'GPL-3.0-or-later.txt': 'GPL-3.0-or-later License text here...'
6767
}
6868
})
69-
const leg = new LicenseEvidenceGatherer({ fs })
69+
const leg = new Contrib.License.Utils.LicenseEvidenceGatherer({ fs })
7070
const found = Array.from(
7171
leg.getFileAttachments('/'))
7272
assert.deepEqual(found, [])
@@ -80,7 +80,7 @@ suite('integration: Utils.LicenseUtility.LicenseEvidenceGatherer', () => {
8080
'GPL-3.0-or-later.txt': 'GPL-3.0-or-later License text here...'
8181
}
8282
})
83-
const leg = new LicenseEvidenceGatherer({ fs })
83+
const leg = new Contrib.License.Utils.LicenseEvidenceGatherer({ fs })
8484
const found = Array.from(
8585
leg.getFileAttachments('/'))
8686
assert.deepEqual(found, [])
@@ -93,7 +93,7 @@ suite('integration: Utils.LicenseUtility.LicenseEvidenceGatherer', () => {
9393
`skipped license file ${sep}LICENSE`,
9494
{ cause: new Error('Custom read error: Access denied!') })
9595
fs.readFileSync = function () { throw expectedError.cause }
96-
const leg = new LicenseEvidenceGatherer({ fs })
96+
const leg = new Contrib.License.Utils.LicenseEvidenceGatherer({ fs })
9797
const errors = []
9898
const found = Array.from(
9999
leg.getFileAttachments(
@@ -132,7 +132,7 @@ suite('integration: Utils.LicenseUtility.LicenseEvidenceGatherer', () => {
132132

133133
test('finds licenses as expected', () => {
134134
const { fs } = memfs({ '/': mockedLicenses })
135-
const leg = new LicenseEvidenceGatherer({ fs })
135+
const leg = new Contrib.License.Utils.LicenseEvidenceGatherer({ fs })
136136
const errors = []
137137
const found = Array.from(
138138
leg.getFileAttachments(
@@ -218,7 +218,7 @@ suite('integration: Utils.LicenseUtility.LicenseEvidenceGatherer', () => {
218218

219219
test('does not find licenses in subfolder', () => {
220220
const { fs } = memfs({ '/foo': mockedLicenses })
221-
const leg = new LicenseEvidenceGatherer({ fs })
221+
const leg = new Contrib.License.Utils.LicenseEvidenceGatherer({ fs })
222222
const found = Array.from(
223223
leg.getFileAttachments('/'))
224224
assert.deepEqual(found, [])

tests/unit/Builders.FromNodePackageJson.ComponentBuilder.spec.js

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -22,19 +22,15 @@ const assert = require('node:assert')
2222
const { suite, test } = require('mocha')
2323

2424
const {
25-
Builders: { FromNodePackageJson: { ComponentBuilder } },
26-
Factories: {
27-
FromNodePackageJson: { ExternalReferenceFactory },
28-
LicenseFactory
29-
}
25+
Contrib,
3026
} = require('../../')
3127

3228
suite('unit: Builders.FromNodePackageJson.ComponentBuilder', () => {
3329
test('construct', () => {
34-
const extRefFactory = new ExternalReferenceFactory()
35-
const licenseFactory = new LicenseFactory()
30+
const extRefFactory = new Contrib.FromNodePackageJson.Factories.ExternalReferenceFactory()
31+
const licenseFactory = new Contrib.License.Factories.LicenseFactory()
3632

37-
const actual = new ComponentBuilder(extRefFactory, licenseFactory)
33+
const actual = new Contrib.FromNodePackageJson.Builders.ComponentBuilder(extRefFactory, licenseFactory)
3834

3935
assert.strictEqual(actual.extRefFactory, extRefFactory)
4036
assert.strictEqual(actual.licenseFactory, licenseFactory)

0 commit comments

Comments
 (0)