Merge pull request #21 from sfaqer/bugfix/gaTests

Исправление ошибок и параметризация версии ovm

Author: nixel2007

.github/workflows/check.yml

@@ -8,14 +8,16 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        oscript_version: [1.3.0, 1.5.0, 1.8.1] # fixme вернуть dev
+        oscript_version: [lts, stable, dev]
         os: [macOS-latest, ubuntu-latest, windows-latest]
+        ovm_version: ['1.2.1', latest]
     name: check oscript
-    steps: 
+    steps:
     - uses: actions/checkout@v2 
     - uses: ./
       with:
         version: ${{ matrix.oscript_version }}
+        ovm_version: ${{ matrix.ovm_version }}
     - name: Проверка скрипта OS
       run: oscript test/fixtures/test.os
     - name: Проверка установки coverage

README.md

@@ -20,7 +20,22 @@ Setup-Onescript позволяет использовать в действия
 * 1.0.21
 * 1.2.0
 * 1.3.0
+* lts (версия длительного сопровождения)
+* stable (последняя выпущенная версия)
 * dev (текущая ночная сборка)
+* preview (версия для ознакомления)
+* lts-dev (версия готовящаяся к выходу в stable)
+
+Так же есть необязательный параметр `ovm_version`
+```yaml
+  with:
+    ovm_version: 1.2.1 # Требуемая версия OVM
+```
+Он отвечает за то какая версия OVM будет использоваться для установки, поддерживает следующие значения:
+* 1.1.0
+* 1.2.0
+* 1.2.1
+* latest (последний выпущенный релиз OVM)
 
 ### Использование matrix
 

action.yml

@@ -7,6 +7,11 @@ author: 'otymko'
 inputs: 
   version:
     description: 'Version of Onescript to use. Support value: 1.3.0, 1.5.0, 1.8.1, dev'
+    required: true
+  ovm_version:
+    description: 'Version of Onescript manager to use. Support value: 1.1.0, 1.2.0, latest'
+    required: false
+    default: 'latest'
 runs:
   using: 'node12'
   main: 'dist/index.js'

dist/index.js

@@ -8657,6 +8657,7 @@ async function run() {
     try {
 
         const osVersion = core.getInput('version');
+        const ovmVersion = core.getInput('ovm_version');
 
         console.log('OS: ' + platform);
 
@@ -8672,7 +8673,12 @@ async function run() {
         if (platform == 'win32') {
             pathToOVM = path.dirname(__dirname) + '/' + 'ovm.exe';
         }
-        await exec.exec('curl -L https://github.com/oscript-library/ovm/releases/download/v1.2.0/ovm.exe --output ' + pathToOVM);
+
+        if (ovmVersion == 'latest') {
+            await exec.exec('curl -L https://github.com/oscript-library/ovm/releases/latest/download/ovm.exe --output ' + pathToOVM);
+        } else {
+            await exec.exec('curl -L https://github.com/oscript-library/ovm/releases/download/v'+ ovmVersion + '/ovm.exe --output ' + pathToOVM);
+        }
 
         if (platform == 'win32') {
             let pathToOVM = path.dirname(__dirname);
@@ -8703,7 +8709,9 @@ async function run() {
         const options = {};
         options.listeners = {
             stdout: (data) => {
-                output += data.toString();
+                if (data.toString().includes('ovm')) {
+                    output += data.toString();
+                }
             }
         };
         await exec.exec('ovm', ['which', 'current'], options);

index.js

@@ -11,6 +11,7 @@ async function run() {
     try {
 
         const osVersion = core.getInput('version');
+        const ovmVersion = core.getInput('ovm_version');
 
         console.log('OS: ' + platform);
 
@@ -26,7 +27,12 @@ async function run() {
         if (platform == 'win32') {
             pathToOVM = path.dirname(__dirname) + '/' + 'ovm.exe';
         }
-        await exec.exec('curl -L https://github.com/oscript-library/ovm/releases/download/v1.2.0/ovm.exe --output ' + pathToOVM);
+
+        if (ovmVersion == 'latest') {
+            await exec.exec('curl -L https://github.com/oscript-library/ovm/releases/latest/download/ovm.exe --output ' + pathToOVM);
+        } else {
+            await exec.exec('curl -L https://github.com/oscript-library/ovm/releases/download/v'+ ovmVersion + '/ovm.exe --output ' + pathToOVM);
+        }
 
         if (platform == 'win32') {
             let pathToOVM = path.dirname(__dirname);
@@ -57,7 +63,9 @@ async function run() {
         const options = {};
         options.listeners = {
             stdout: (data) => {
-                output += data.toString();
+                if (data.toString().includes('ovm')) {
+                    output += data.toString();
+                }
             }
         };
         await exec.exec('ovm', ['which', 'current'], options);