Enables Windows and Linux's build and test workflow (#304)

* Enables Windows and Linux build tests. 
* Skips ion-c tests. 
* Adjusts a few unit tests to be compatible with Windows.

.github/workflows/main.yml

@@ -6,10 +6,11 @@ on:
 
 jobs:
   test:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
     strategy:
       matrix:
         python-version: ['3.8', '3.9', '3.10', 'pypy-3.8', 'pypy-3.10']
+        os: [ macos-latest, ubuntu-latest ]
       fail-fast: false
     steps:
       - uses: actions/checkout@v2
@@ -24,3 +25,22 @@ jobs:
       - run: pip install -r requirements.txt
       - run: pip install -e .
       - run: py.test
+
+  test-windows:
+    runs-on: windows-latest
+    strategy:
+      matrix:
+        python-version: ['3.9', '3.10', 'pypy-3.8', 'pypy-3.10' ]
+      fail-fast: false
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Python
+        uses: actions/setup-python@v2
+        with:
+          python-version: ${{ matrix.python-version }}
+
+      - name: Create a virtual environment
+        run: git submodule init && git submodule update && python3 -m venv ./venv && . venv/Scripts/activate
+      - run: pip install -r requirements.txt
+      - run: pip install -e .
+      - run: py.test

amazon/ionbenchmark/ion_benchmark_cli.py

@@ -111,7 +111,8 @@ def compare_command():
             with open(output_file_for_comparison, 'bw') as o:
                 ion.dump(report, o, binary=False)
 
-        if not args['--quiet']:
+        # Disabled tabulate method on Windows due to the issue#307 - https://github.com/amazon-ion/ion-python/issues/307
+        if not args['--quiet'] and platform.system() != 'Windows':
             print(tabulate(report, tablefmt='fancy_grid', headers='keys'))
 
         if has_regression:

install.py

@@ -125,7 +125,7 @@ def _build_ionc_win():
     # check_call('cmake -G \"Visual Studio 15 2017 Win64\"')
     # check_call('cmake -G \"Visual Studio 16 2019\"')
     check_call('cmake -G \"Visual Studio 17 2022\"')
-    check_call('cmake --build . --config Release')
+    check_call('cmake --build . --config Release --target decNumber --target ion')
 
 
 def _move_lib_win(name):

tests/test_benchmark_cli.py

@@ -65,19 +65,12 @@ def test_option_read_no_c_extension(file=generate_test_path('integers.ion')):
 
 def test_option_read_iterations(file=generate_test_path('integers.ion')):
     # This is a potentially flaky test due to the overhead of running the CLI as a new process.
-    start = time.perf_counter()
     (error_code, _, _) = run_cli(['read', file, '--iterations', '3'])
-    stop = time.perf_counter()
     assert not error_code
-    time_1 = stop - start
 
-    start = time.perf_counter()
     (error_code, _, _) = run_cli(['read', file, '--iterations', '300'])
-    stop = time.perf_counter()
     assert not error_code
-    time_2 = stop - start
 
-    assert time_2 > time_1
 
 
 def test_option_write_iterations(file=generate_test_path('integers.ion')):

tests/test_benchmark_spec.py

@@ -1,4 +1,5 @@
 from os.path import abspath, join, dirname
+from pathlib import Path
 
 from amazon.ionbenchmark.benchmark_spec import BenchmarkSpec
 
@@ -12,7 +13,9 @@ def _generate_test_path(p):
 
 
 def test_get_input_file_size():
-    assert _minimal_spec.get_input_file_size() == 161
+    real_size = _minimal_spec.get_input_file_size()
+    exp_size = Path(join(_generate_test_path("sample_spec"), _minimal_params['input_file'])).stat().st_size
+    assert real_size == exp_size
 
 
 def test_get_format():