refactor: use camelCase for filenames

lib/compile.js

@@ -1,6 +1,6 @@
-const detectTransactionExampleNumbers = require('./detect-transaction-example-numbers');
-const compileUri = require('./compile-uri');
-const getTransactionName = require('./transaction-name');
+const detectTransactionExampleNumbers = require('./detectTransactionExampleNumbers');
+const compileUri = require('./compileURI');
+const compileTransactionName = require('./compileTransactionName');
 
 
 function findRelevantTransactions(mediaType, apiElements) {
@@ -157,7 +157,7 @@ function compileTransaction(mediaType, filename, httpTransactionElement, example
 
   if (!request) { return { transaction: null, annotations }; }
 
-  const name = getTransactionName(origin);
+  const name = compileTransactionName(origin);
   const response = compileResponse(httpTransactionElement.response);
 
   const transaction = {

lib/transaction-name/index.js → lib/compileTransactionName.js

@@ -1,4 +1,4 @@
-module.exports = function index(origin) {
+module.exports = function compileTransactionName(origin) {
   const segments = [];
   if (origin.apiName) { segments.push(origin.apiName); }
   if (origin.resourceGroupName) { segments.push(origin.resourceGroupName); }

lib/compile-uri/index.js → lib/compileURI/index.js

@@ -1,8 +1,9 @@
-const compileParams = require('./compile-params');
-const validateParams = require('./validate-params');
-const expandUriTemplate = require('./expand-uri-template');
+const compileParams = require('./compileParams');
+const validateParams = require('./validateParams');
+const expandUriTemplate = require('./expandURItemplate');
 
-module.exports = function index(httpRequestElement) {
+
+module.exports = function compileURI(httpRequestElement) {
   const annotations = [];
   const cascade = [
     httpRequestElement.parents.find('resource'),

test/integration/compile-test.js

@@ -1,7 +1,7 @@
 const proxyquire = require('proxyquire').noPreserveCache();
 
-const createAnnotationSchema = require('../schemas/annotation');
-const createCompileResultSchema = require('../schemas/compile-result');
+const createAnnotationSchema = require('../schemas/createAnnotationSchema');
+const createCompileResultSchema = require('../schemas/createCompileResultSchema');
 
 const { assert, fixtures } = require('../support');
 const compile = require('../../lib/compile');
@@ -147,8 +147,8 @@ describe('compile() · all API description formats', () => {
     fixtures('ordinary').forEachDescribe(({ mediaType, apiElements }) => {
       const message = '... dummy warning message ...';
       const stubbedCompile = proxyquire('../../lib/compile', {
-        './compile-uri': proxyquire('../../lib/compile-uri', {
-          './expand-uri-template': () => ({ uri: '/honey?beekeeper=Honza', errors: [], warnings: [message] }),
+        './compileURI': proxyquire('../../lib/compileURI', {
+          './expandURItemplate': () => ({ uri: '/honey?beekeeper=Honza', errors: [], warnings: [message] }),
         }),
       });
       const compileResult = stubbedCompile(mediaType, apiElements);
@@ -208,8 +208,8 @@ describe('compile() · all API description formats', () => {
     fixtures('ordinary').forEachDescribe(({ mediaType, apiElements }) => {
       const message = '... dummy warning message ...';
       const stubbedCompile = proxyquire('../../lib/compile', {
-        './compile-uri': proxyquire('../../lib/compile-uri', {
-          './validate-params': () => ({ errors: [], warnings: [message] }),
+        './compileURI': proxyquire('../../lib/compileURI', {
+          './validateParams': () => ({ errors: [], warnings: [message] }),
         }),
       });
       const compileResult = stubbedCompile(mediaType, apiElements);

test/integration/compile-apib-test.js → test/integration/compileAPIB-test.js

@@ -1,9 +1,9 @@
 const sinon = require('sinon');
 const proxyquire = require('proxyquire').noPreserveCache();
 
-const createAnnotationSchema = require('../schemas/annotation');
-const createCompileResultSchema = require('../schemas/compile-result');
-const detectTransactionExampleNumbers = require('../../lib/detect-transaction-example-numbers');
+const createAnnotationSchema = require('../schemas/createAnnotationSchema');
+const createCompileResultSchema = require('../schemas/createCompileResultSchema');
+const detectTransactionExampleNumbers = require('../../lib/detectTransactionExampleNumbers');
 
 const { assert, fixtures } = require('../support');
 const compile = require('../../lib/compile');
@@ -53,7 +53,7 @@ describe('compile() · API Blueprint', () => {
   describe('with multiple transaction examples', () => {
     const detectTransactionExampleNumbersStub = sinon.spy(detectTransactionExampleNumbers);
     const stubbedCompile = proxyquire('../../lib/compile', {
-      './detect-transaction-example-numbers': detectTransactionExampleNumbersStub,
+      './detectTransactionExampleNumbers': detectTransactionExampleNumbersStub,
     });
     const { mediaType, apiElements } = fixtures('multiple-transaction-examples').apib;
     const compileResult = stubbedCompile(mediaType, apiElements);
@@ -100,7 +100,7 @@ describe('compile() · API Blueprint', () => {
   describe('without multiple transaction examples', () => {
     const detectTransactionExampleNumbersStub = sinon.spy(detectTransactionExampleNumbers);
     const stubbedCompile = proxyquire('../../lib/compile', {
-      './detect-transaction-example-numbers': detectTransactionExampleNumbersStub,
+      './detectTransactionExampleNumbers': detectTransactionExampleNumbersStub,
     });
     const { mediaType, apiElements } = fixtures('one-transaction-example').apib;
     const compileResult = stubbedCompile(mediaType, apiElements);

test/integration/compile-openapi2-test.js → test/integration/compileOpenAPI2-test.js

@@ -1,9 +1,9 @@
 const sinon = require('sinon');
 const proxyquire = require('proxyquire');
 
-const createAnnotationSchema = require('../schemas/annotation');
-const createCompileResultSchema = require('../schemas/compile-result');
-const detectTransactionExampleNumbers = require('../../lib/detect-transaction-example-numbers');
+const createAnnotationSchema = require('../schemas/createAnnotationSchema');
+const createCompileResultSchema = require('../schemas/createCompileResultSchema');
+const detectTransactionExampleNumbers = require('../../lib/detectTransactionExampleNumbers');
 
 const { assert, fixtures } = require('../support');
 const compile = require('../../lib/compile');
@@ -136,7 +136,7 @@ describe('compile() · OpenAPI 2', () => {
     const filename = 'apiDescription.json';
     const detectTransactionExampleNumbersStub = sinon.spy(detectTransactionExampleNumbers);
     const stubbedCompile = proxyquire('../../lib/compile', {
-      './detect-transaction-example-numbers': detectTransactionExampleNumbersStub,
+      './detectTransactionExampleNumbers': detectTransactionExampleNumbersStub,
     });
     const { mediaType, apiElements } = fixtures('multiple-responses').openapi2;
     const compileResult = stubbedCompile(mediaType, apiElements, filename);

test/integration/compile-openapi3-test.js → test/integration/compileOpenAPI3-test.js

@@ -1,4 +1,4 @@
-const createCompileResultSchema = require('../schemas/compile-result');
+const createCompileResultSchema = require('../schemas/createCompileResultSchema');
 
 const { assert, fixtures } = require('../support');
 const compile = require('../../lib/compile');

test/integration/dredd-transactions-test.js → test/integration/dreddTransactions-test.js

@@ -1,11 +1,12 @@
 const proxyquire = require('proxyquire').noPreserveCache();
 
-const createCompileResultSchema = require('../schemas/compile-result');
-const createAnnotationSchema = require('../schemas/annotation');
+const createCompileResultSchema = require('../schemas/createCompileResultSchema');
+const createAnnotationSchema = require('../schemas/createAnnotationSchema');
 const dreddTransactions = require('../../lib/index');
 
 const { assert, fixtures } = require('../support');
 
+
 describe('Dredd Transactions', () => {
   describe('when compilation throws an exception', () => {
     const error = new Error('... dummy message ...');

test/schemas/annotation.js → test/schemas/createAnnotationSchema.js

@@ -1,9 +1,10 @@
-const createLocationSchema = require('./location');
-const createOriginSchema = require('./origin');
+const createLocationSchema = require('./createLocationSchema');
+const createOriginSchema = require('./createOriginSchema');
 
 const TYPES = ['error', 'warning'];
 const COMPONENTS = ['apiDescriptionParser', 'parametersValidation', 'uriTemplateExpansion'];
 
+
 module.exports = function createAnnotationSchema(options = {}) {
   // Either filename string or undefined (= doesn't matter)
   const { filename } = options;

test/schemas/compile-result.js → test/schemas/createCompileResultSchema.js

@@ -1,5 +1,6 @@
-const createOriginSchema = require('./origin');
-const createAnnotationSchema = require('./annotation');
+const createOriginSchema = require('./createOriginSchema');
+const createAnnotationSchema = require('./createAnnotationSchema');
+
 
 function addMinMax(schema, n) {
   const modifiedSchema = Object.assign({}, schema);
@@ -15,6 +16,7 @@ function addMinMax(schema, n) {
   return modifiedSchema;
 }
 
+
 module.exports = function createCompileResultSchema(options = {}) {
   // Either filename string or undefined (= doesn't matter)
   const { filename } = options;

test/unit/transaction-name/index-test.js → test/unit/compileTransactionName-test.js

@@ -1,12 +1,13 @@
 const { assert } = require('chai');
 
-const getTransactionName = require('../../../lib/transaction-name');
+const compileTransactionName = require('../../lib/compileTransactionName');
 
-describe('getTransactionName()', () => {
-  it('is a function', () => assert.isFunction(getTransactionName));
+
+describe('compileTransactionName()', () => {
+  it('is a function', () => assert.isFunction(compileTransactionName));
 
   it('joins all parts of the origin object', () => {
-    const name = getTransactionName({
+    const name = compileTransactionName({
       apiName: 'a',
       resourceGroupName: 'b',
       resourceName: 'c',
@@ -17,7 +18,7 @@ describe('getTransactionName()', () => {
   });
 
   it('joins just the parts of the origin object, which are available', () => {
-    const name = getTransactionName({
+    const name = compileTransactionName({
       apiName: null,
       resourceGroupName: 'a',
       resourceName: undefined,
@@ -28,7 +29,7 @@ describe('getTransactionName()', () => {
   });
 
   it('returns no separators if the origin object contains just one part', () => {
-    const name = getTransactionName({
+    const name = compileTransactionName({
       apiName: null,
       resourceGroupName: 'a',
       resourceName: undefined,
@@ -39,7 +40,7 @@ describe('getTransactionName()', () => {
   });
 
   it('does not mind if any part of the origin object already contains the separator', () => {
-    const name = getTransactionName({
+    const name = compileTransactionName({
       apiName: 'a',
       resourceGroupName: 'b',
       resourceName: 'c',

test/unit/compile-uri/compile-params-test.js → test/unit/compileURI/compileParams-test.js

@@ -1,7 +1,7 @@
 const { assert } = require('chai');
 const fury = require('fury');
 
-const compileParams = require('../../../lib/compile-uri/compile-params');
+const compileParams = require('../../../lib/compileURI/compileParams');
 
 
 describe('compileParams()', () => {

test/unit/compile-uri/expand-uri-template-test.js → test/unit/compileURI/expandURItemplate-test.js

@@ -1,6 +1,7 @@
 const { assert } = require('chai');
 
-const expandUriTemplate = require('../../../lib/compile-uri/expand-uri-template');
+const expandUriTemplate = require('../../../lib/compileURI/expandURItemplate');
+
 
 describe('expandUriTemplate()', () => {
   let data = null;

test/unit/compile-uri/validate-parameters-test.js → test/unit/compileURI/validateParams-test.js

@@ -1,6 +1,7 @@
 const { assert } = require('chai');
 
-const validateParams = require('../../../lib/compile-uri/validate-params');
+const validateParams = require('../../../lib/compileURI/validateParams');
+
 
 describe('validateParams()', () => {
   it('should return an object', () => {

test/unit/detect-transaction-example-numbers-test.js → test/unit/detectTransactionExampleNumbers-test.js

@@ -1,4 +1,4 @@
-const detectTransactionExampleNumbers = require('../../lib/detect-transaction-example-numbers');
+const detectTransactionExampleNumbers = require('../../lib/detectTransactionExampleNumbers');
 const parse = require('../../lib/parse');
 
 const { assert } = require('../support');