Skip to content

Commit

Permalink
Fix URIs for V3 (#40)
Browse files Browse the repository at this point in the history
The version 3 of the AAS meta-model removed the `www.` prefix in the
URIs used to navigate the aasx package file.

Fixes #39.

Since the samples available on the web site are incorrect according to
the specification, we remove the tests related to the official AASX
samples.
  • Loading branch information
mristin authored Mar 8, 2024
1 parent ed53a53 commit c13d250
Show file tree
Hide file tree
Showing 9 changed files with 13 additions and 704 deletions.
4 changes: 0 additions & 4 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,6 @@ jobs:
working-directory: src
run: dotnet tool install coveralls.net --version 2.0.0-beta0002

- name: Download samples
working-directory: src
run: powershell .\DownloadSamples.ps1

- name: Run tests
working-directory: src
run: powershell .\Test.ps1
Expand Down
3 changes: 0 additions & 3 deletions doc/contributing/continuous-integration.md
Original file line number Diff line number Diff line change
Expand Up @@ -62,9 +62,6 @@ Please see the source code of `src\Check.ps1` for more details.
Our tests with external dependencies use environment variables to specify the location of the dependencies.
While `src\Check.ps1` and related scripts set up the expected locations in the environment automatically, you need to adjust your test setting accordingly.

So far, we need to set `SAMPLE_AASX_DIR` environment variable to point to the absolute path where AASX samples have been downloaded (using `src\DownloadSamples.ps1`).
This is by definition `{your repository}\sample-aasx`.

## Github Workflows

Github Actions allow for running continuous integration on Github servers.
Expand Down
70 changes: 0 additions & 70 deletions src/AasCore.Aas3.Package.Tests/SampleAasxDir.cs

This file was deleted.

138 changes: 0 additions & 138 deletions src/AasCore.Aas3.Package.Tests/Table.cs

This file was deleted.

Loading

0 comments on commit c13d250

Please sign in to comment.