v1.2.1 has carriage return in bin/cli.js shebang line #144 (#46)

* Add test for CLI execution #44

* Set cli.js to be always have lf line endings #44, update files to comply to .editorconfig

.gitattributes

@@ -0,0 +1 @@
+/bin/cli.js text eol=lf

.gitignore

@@ -13,4 +13,4 @@ Thumbs.db
 # Node / npm
 .npm
 /node_modules/
-/package-lock.json
+/package-lock.json

bin/cli.js

@@ -1,2 +1,2 @@
 #!/usr/bin/env node
-require('../index.js')()
+require('../index.js')()

index.js

@@ -409,4 +409,4 @@ async function loadSchema(schemaId) {
 
 module.exports = async config => {
 	await run(config);
-};
+};

tests/api/collections.json

@@ -78,4 +78,4 @@
     }
   ],
 	"links": []
-}
+}

tests/api/items.json

@@ -80,4 +80,4 @@
 
 	],
 	"links": []
-}
+}

tests/cli.test.js

@@ -1,6 +1,7 @@
 const app = require('../index');
 const { version } = require('../package.json');
 const fs = require('fs/promises');
+const { exec } = require("child_process");
 
 let consoleErrSpy, consoleWarnSpy, consoleInfSpy, consoleLogSpy, mockExit;
 const initString = `STAC Node Validator v${version}`;
@@ -20,6 +21,15 @@ beforeEach(() => {
 	consoleErrSpy = jest.spyOn(console, 'error').mockImplementation();
 });
 
+it('Should be executable via shell', done => {
+	exec("node ./bin/cli.js --version", (error, stdout, stderr) => {
+		expect(error).toBe(null);
+		expect(stderr).toBe("");
+		expect(stdout).toContain(version);
+		done();
+	});
+});
+
 it('Should print init string', async () => {
 	await app();
 

tests/example-config.json

@@ -1,4 +1,4 @@
 {
   "files": ["tests/examples/catalog.json"],
   "verbose": true
-}
+}

tests/examples/catalog.json

@@ -5,4 +5,4 @@
 	"title": "Example Catalog",
 	"description": "A minimal example for tests",
 	"links": []
-}
+}

tests/examples/invalid-catalog.json

@@ -4,4 +4,4 @@
 	"type": "Catalog",
 	"title": "Example Catalog",
 	"description": "A minimal example for tests"
-}
+}