updated lib

README.md

@@ -6,11 +6,11 @@ This action sets up a flutter environment for use in actions. It works on Linux,
 
 ```yaml
 steps:
-- uses: actions/checkout@master
+- uses: actions/checkout@v1
 - uses: actions/setup-java@v1
   with:
     java-version: '12.x'
-- uses: subosito/flutter-action@master
+- uses: subosito/flutter-action@v1
   with:
     version: '1.7.8+hotfix.4'
 - run: flutter pub get
@@ -29,11 +29,11 @@ jobs:
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
     steps:
-      - uses: actions/checkout@master
-      - uses: actions/setup-java@v1.0.1
+      - uses: actions/checkout@v1
+      - uses: actions/setup-java@v1
         with:
           java-version: '12.x'
-      - uses: subosito/flutter-action@master
+      - uses: subosito/flutter-action@v1
         with:
           version: '1.7.8+hotfix.4'
       - run: flutter pub get

__tests__/installer.test.ts

@@ -29,7 +29,7 @@ describe('installer tests', () => {
     await installer.getFlutter('1.7.8+hotfix.4', 'stable');
     const sdkDir = path.join(
       toolDir,
-      'Flutter',
+      'flutter',
       '1.7.8-hotfix.4-stable',
       'x64'
     );
@@ -40,9 +40,19 @@ describe('installer tests', () => {
 
   it('Downloads flutter from beta channel', async () => {
     await installer.getFlutter('1.8.3', 'beta');
-    const sdkDir = path.join(toolDir, 'Flutter', '1.8.3-beta', 'x64');
+    const sdkDir = path.join(toolDir, 'flutter', '1.8.3-beta', 'x64');
 
     expect(fs.existsSync(`${sdkDir}.complete`)).toBe(true);
     expect(fs.existsSync(path.join(sdkDir, 'bin'))).toBe(true);
   }, 100000);
+
+  it('Throws if no location contains correct flutter version', async () => {
+    let thrown = false;
+    try {
+      await installer.getFlutter('1000.0', 'dev');
+    } catch {
+      thrown = true;
+    }
+    expect(thrown).toBe(true);
+  });
 });

lib/installer.js

@@ -49,7 +49,7 @@ function getFlutter(version, channel) {
         // make semver compatible, eg: 1.7.8+hotfix.4 -> 1.7.8-hotfix.4
         const semver = version.replace('+', '-');
         const cleanver = `${semver}-${channel}`;
-        let toolPath = tc.find('Flutter', cleanver);
+        let toolPath = tc.find('flutter', cleanver);
         if (toolPath) {
             core.debug(`Tool found in cache ${toolPath}`);
         }
@@ -60,7 +60,7 @@ function getFlutter(version, channel) {
             let tempDir = generateTempDir();
             const sdkDir = yield extractDownload(sdkFile, tempDir);
             core.debug(`Flutter sdk extracted to ${sdkDir}`);
-            toolPath = yield tc.cacheDir(sdkDir, 'Flutter', cleanver);
+            toolPath = yield tc.cacheDir(sdkDir, 'flutter', cleanver);
         }
         core.exportVariable('FLUTTER_HOME', toolPath);
         core.addPath(path.join(toolPath, 'bin'));

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "flutter-action",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,8 +1,8 @@
 {
   "name": "flutter-action",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "private": true,
-  "description": "flutter action",
+  "description": "Flutter environment for use in actions",
   "main": "lib/index.js",
   "scripts": {
     "build": "tsc",

src/installer.ts

@@ -35,7 +35,7 @@ export async function getFlutter(
   // make semver compatible, eg: 1.7.8+hotfix.4 -> 1.7.8-hotfix.4
   const semver = version.replace('+', '-');
   const cleanver = `${semver}-${channel}`;
-  let toolPath = tc.find('Flutter', cleanver);
+  let toolPath = tc.find('flutter', cleanver);
 
   if (toolPath) {
     core.debug(`Tool found in cache ${toolPath}`);
@@ -49,7 +49,7 @@ export async function getFlutter(
     const sdkDir = await extractDownload(sdkFile, tempDir);
     core.debug(`Flutter sdk extracted to ${sdkDir}`);
 
-    toolPath = await tc.cacheDir(sdkDir, 'Flutter', cleanver);
+    toolPath = await tc.cacheDir(sdkDir, 'flutter', cleanver);
   }
 
   core.exportVariable('FLUTTER_HOME', toolPath);