switch none deployment strategy to basic (#204)

* switch none deployment strategy to basic

* update readme

* update deployment strategy fallthrough logic

* comment fixed

* add disclaimer for basic strategy only supporting deploy action

src/strategyHelpers/deploymentHelper.ts

@@ -70,10 +70,7 @@ export async function deployManifests(
          return newFilePaths
       }
 
-      case undefined: {
-         core.warning('Deployment strategy is not recognized.')
-      }
-      default: {
+      case DeploymentStrategy.BASIC: {
          const trafficSplitMethod = parseTrafficSplitMethod(
             core.getInput('traffic-split-method', {required: true})
          )
@@ -94,6 +91,10 @@ export async function deployManifests(
 
          return files
       }
+
+      default: {
+         throw new Error('Deployment strategy is not recognized.')
+      }
    }
 }
 

src/types/deploymentStrategy.test.ts

@@ -5,6 +5,7 @@ describe('Deployment strategy type', () => {
       const vals = <any>Object.values(DeploymentStrategy)
       expect(vals.includes('canary')).toBe(true)
       expect(vals.includes('blue-green')).toBe(true)
+      expect(vals.includes('basic')).toBe(true)
    })
 
    test('it can parse valid values from a string', () => {

src/types/deploymentStrategy.ts

@@ -1,4 +1,5 @@
 export enum DeploymentStrategy {
+   BASIC = 'basic',
    CANARY = 'canary',
    BLUE_GREEN = 'blue-green'
 }