chore(config-tests): refactor config tests (#1102)

* chore(config-tests): refactor config tests

* chore(config-tests): adjust tests to pick up .contentfulrc

test/unit/bin/lib/config.spec.ts

@@ -8,17 +8,44 @@ const fileConfig = {
   cmaToken: process.env.CONTENTFUL_INTEGRATION_TEST_CMA_TOKEN
 }
 
+function deleteRequireCache() {
+  Object.keys(require.cache).forEach((key) => {
+    delete require.cache[key]
+  })
+}
+
 describe('Config', function () {
   beforeEach(function () {
+    deleteRequireCache()
     writeFileSync(resolve('/tmp', '.contentfulrc.json'), JSON.stringify(fileConfig))
   })
 
-  it('reads the contentfulrc.json', async function () {
+  it('reads the contentfulrc.json', function () {
     const config = getConfig({})
     expect(config.accessToken).to.eql(process.env.CONTENTFUL_INTEGRATION_TEST_CMA_TOKEN)
   })
 
-  it('prefers env config over contentfulrc.json', async function () {
+  it('chooses cmaToken even if managementToken is provided as well', function () {
+    writeFileSync(
+      resolve('/tmp', '.contentfulrc.json'),
+      JSON.stringify({ cmaToken: 'cmaToken', managementToken: 'managementToken' })
+    )
+    const config = getConfig({})
+    expect(config.accessToken).to.eql('cmaToken')
+  })
+
+  // This behavior may be adjusted in the future as some tools which wrap contentful-migration use
+  // managementToken instead of cmaToken and we may want it to be picked up to ensure compatibility
+  it('does not pick up managementToken even if no cmaToken is provided', function () {
+    writeFileSync(
+      resolve('/tmp', '.contentfulrc.json'),
+      JSON.stringify({ managementToken: 'managementToken' })
+    )
+    const config = getConfig({})
+    expect(config.accessToken).to.eql(undefined)
+  })
+
+  it('prefers env config over contentfulrc.json', function () {
     const token = process.env.CONTENTFUL_INTEGRATION_TEST_CMA_TOKEN
 
     process.env.CONTENTFUL_MANAGEMENT_ACCESS_TOKEN = 'schnitzel'
@@ -27,7 +54,7 @@ describe('Config', function () {
     process.env.CONTENTFUL_INTEGRATION_TEST_CMA_TOKEN = token
   })
 
-  it('prefers handed in config over env config', async function () {
+  it('prefers handed in config over env config', function () {
     const config = getConfig({ accessToken: 'fooMyBar' })
     expect(config.accessToken).to.eql('fooMyBar')
   })