Merge pull request #2717 from SciML/os/fix-tests

fix even more ci?

Author: ChrisRackauckas

.buildkite/pipeline.yml

@@ -3,13 +3,6 @@ steps:
     plugins:
       - JuliaCI/julia#v1:
           version: "1"
-      - staticfloat/metahook:
-          pre-command: |
-            julia --project=. -e '
-              using Pkg;
-              VERSION < v"1.11" && Pkg.develop(map(path ->Pkg.PackageSpec.(;path="$(@__DIR__)/lib/$(path)"), readdir("./lib")));
-              Pkg.instantiate();
-            '
       - JuliaCI/julia-test#v1:
            coverage: false # 1000x slowdown
     agents:
@@ -41,13 +34,6 @@ steps:
       JULIA_NUM_THREADS: 2
     plugins:
       - JuliaCI/julia#v1
-      - staticfloat/metahook:
-          pre-command: |
-            julia --project=. -e '
-              using Pkg;
-              VERSION < v"1.11" && Pkg.develop(map(path ->Pkg.PackageSpec.(;path="$(@__DIR__)/lib/$(path)"), readdir("./lib")));
-              Pkg.instantiate();
-            '
       - JuliaCI/julia-test#v1:
           coverage: false
     agents:

.github/workflows/CI.yml

@@ -78,15 +78,6 @@ jobs:
             ${{ runner.os }}-test-${{ env.cache-name }}-
             ${{ runner.os }}-test-
             ${{ runner.os }}-
-      # Explicitly develop the libraries first before running the tests for now.
-      # This is necessary since the tests are likely to fail otherwise, given that all
-      # the libs haven't been registered yet.
-      - name: "Develop the libraries since they haven't been registered yet"
-        run: |
-          julia --project=. -e '
-              using Pkg;
-              VERSION < v"1.11" && Pkg.develop(map(path ->Pkg.PackageSpec.(;path="$(@__DIR__)/lib/$(path)"), readdir("./lib")));
-          '
       - uses: julia-actions/julia-runtest@v1
         with:
           coverage: false

.github/workflows/Documentation.yml

@@ -15,15 +15,6 @@ jobs:
       - uses: julia-actions/setup-julia@latest
         with:
           version: '1'
-      # Explicitly develop the libraries first before running the tests for now.
-      # This is necessary since the tests are likely to fail otherwise, given that all
-      # the libs haven't been registered yet.
-      - name: "Develop the libraries since they haven't been registered yet"
-        run: |
-          julia --project=. -e '
-              using Pkg;
-              Pkg.develop(map(path ->Pkg.PackageSpec.(;path="$(@__DIR__)/lib/$(path)"), readdir("./lib")));
-          '
       - name: Install dependencies
         run: julia --project=docs/ -e 'using Pkg; Pkg.develop(PackageSpec(path=pwd())); Pkg.instantiate()'
       - name: Build and deploy

.github/workflows/Downstream.yml

@@ -44,32 +44,11 @@ jobs:
         with:
           version: ${{ matrix.julia-version }}
           arch: x64
-      # Explicitly develop the libraries first before running the tests for now.
-      # This is necessary since the tests are likely to fail otherwise, given that all
-      # the libs haven't been registered yet.
       - name: Clone Downstream
         uses: actions/checkout@v4
         with:
           repository: ${{ matrix.package.user }}/${{ matrix.package.repo }}
           path: downstream
-      - name: Load this and run the downstream tests
-        shell: julia --color=yes --project=downstream {0}
-        run: |
-          using Pkg
-          try
-            # force it to use this PR's version of the package
-            VERSION < v"1.11" && Pkg.develop(map(path ->Pkg.PackageSpec.(;path="lib/$(path)"), readdir("./lib")));
-            VERSION < v"1.11" && Pkg.develop(PackageSpec(path="."))  # resolver may fail with main deps
-            Pkg.update()
-            Pkg.test(coverage=true)  # resolver may fail with test time deps
-          catch err
-            err isa Pkg.Resolve.ResolverError || rethrow()
-            # If we can't resolve that means this is incompatible by SemVer and this is fine
-            # It means we marked this as a breaking change, so we don't need to worry about
-            # mistakenly introducing a breaking change, as we have intentionally made one
-            @info "Not compatible with this release. No problem." exception=err
-            exit(0)  # Exit immediately, as a success
-          end
       - uses: julia-actions/julia-processcoverage@v1
       - uses: codecov/codecov-action@v5
         with:

.github/workflows/Invalidations.yml

@@ -20,15 +20,6 @@ jobs:
       with:
         version: '1'
     - uses: actions/checkout@v4
-    # Explicitly develop the libraries first before running the tests for now.
-    # This is necessary since the tests are likely to fail otherwise, given that all
-    # the libs haven't been registered yet.
-    - name: "Develop the libraries since they haven't been registered yet"
-      run: |
-        julia --project=. -e '
-            using Pkg;
-            VERSION < v"1.11" && Pkg.develop(map(path ->Pkg.PackageSpec.(;path="$(@__DIR__)/lib/$(path)"), readdir("./lib")));
-        '
     - uses: julia-actions/julia-invalidations@v1
       id: invs_pr
 

test/runtests.jl

@@ -33,8 +33,8 @@ end
 
 @time begin
     if contains(GROUP, "OrdinaryDiffEq") || GROUP == "ImplicitDiscreteSolve" || GROUP == "SimpleImplicitDiscreteSolve"
-        Pkg.develop(path = "../lib/$GROUP")
-        Pkg.test(GROUP)
+        Pkg.activate(joinpath(dirname(@__DIR__), "lib", GROUP))
+        Pkg.test(GROUP, julia_args=["--check-bounds=auto", "--compiled-modules=yes", "--depwarn=yes"], force_latest_compatible_version=false, allow_reresolve=true)
     elseif GROUP == "All" || GROUP == "InterfaceI" || GROUP == "Interface"
         @time @safetestset "Discrete Algorithm Tests" include("interface/discrete_algorithm_test.jl")
         @time @safetestset "Tstops Tests" include("interface/ode_tstops_tests.jl")