tests(compute-static): remove integration test using hacky Python script

tests(compute-static): add specific test for http method filter function
ci(compute-static): install cargo-nextest and viceroy via taiki-e/install-action
chore(compute-static): revert toolchain back to stable since fastly compute now works again on Rust 1.91.1

Author: Carbonhell

.github/workflows/main.yml

@@ -14,6 +14,11 @@ jobs:
       - name: Install Rust stable
         run: rustup update stable && rustup default stable && rustup component add clippy rustfmt
 
+      - name: Install tools for compute-static tests
+        uses: taiki-e/install-action@493d7f216ecab2af0602481ce809ab2c72836fa1 # v2.62.62
+        with:
+          tool: cargo-nextest,viceroy
+
       - name: Run rustfmt
         run: cargo fmt --all -- --check
 
@@ -25,10 +30,7 @@ jobs:
 
       - name: Run compute-static tests
         working-directory: terragrunt/modules/crates-io/compute-static
-        run: |
-          curl -LsSf https://get.nexte.st/latest/linux | tar zxf - -C ${CARGO_HOME:-~/.cargo}/bin
-          cargo install --locked viceroy --version 0.15.0
-          ./scripts/run_tests.sh
+        run: cargo nextest run
 
   terraform:
     name: Terraform configuration

terragrunt/modules/crates-io/compute-static/README.md

@@ -34,15 +34,10 @@ Then, install [Viceroy](https://github.com/fastly/Viceroy) to run the edge funct
 cargo install --locked viceroy
 ```
 
-Due to the fact Viceroy does not allow easily mocking HTTP requests being sent (
-see [issue](https://github.com/fastly/Viceroy/issues/442)), some tests use a small Python HTTP
-server to work.
-For this reason, a wrapper bash script is provided that runs `cargo nextest run` with the test server active in
-background. You can therefore run the tests with:
-:
+Finally, run the tests:
 
 ```shell
-./run_tests.sh
+cargo nextest run
 ```
 
 ## Deployment

terragrunt/modules/crates-io/compute-static/rust-toolchain.toml

@@ -1,3 +1,3 @@
 [toolchain]
-channel = "1.90"
+channel = "stable"
 targets = ["wasm32-wasip1"]

terragrunt/modules/crates-io/compute-static/scripts/run_tests.sh

@@ -245,8 +245,6 @@ fn http_version_to_string(version: Version) -> Option<String> {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use fastly::experimental::BackendExt;
-    use fastly::Backend;
 
     fn new_test_config() -> Config {
         Config {
@@ -324,21 +322,21 @@ mod tests {
         );
     }
 
-    /// Ensures that the request is forwarded to the primary host for normal queries.
-    /// Assumes the tests are being executed with the test HTTP Python server active in background (see README.md).
+    /// Ensures that only GET and HEAD requests are allowed, and that other methods return .
     #[test]
-    fn test_handle_request() {
-        let config = new_test_config();
-        Backend::builder(&config.primary_host, &config.primary_host)
-            .finish()
-            .unwrap();
-        let client_req = Request::get(
-            "https://static.crates.io/crates/libgit2-sys/libgit2-sys-0.12.25+1.3.0.crate",
-        );
-        let mut res = handle_request(&config, client_req).unwrap();
-        assert_eq!(res.get_status(), StatusCode::OK);
-        // Assuming the function sent a request to the primary host, verify whether the body is the one set in the test server
-        let body = res.take_body_str();
-        assert_eq!(body, "test_data");
+    fn test_limit_http_methods() {
+        let whitelist = [Method::GET, Method::HEAD];
+        let blacklist = [Method::POST, Method::PATCH, Method::PUT, Method::DELETE];
+        for method in whitelist {
+            let client_req = Request::new(method, "http://example.com");
+            assert!(limit_http_methods(&client_req).is_none());
+        }
+        for method in blacklist {
+            let client_req = Request::new(method, "http://example.com");
+            let res = limit_http_methods(&client_req);
+            assert!(res.is_some());
+            // Ensure parity between CloudFront and Fastly
+            assert_eq!(res.unwrap().get_status(), StatusCode::UNAUTHORIZED);
+        }
     }
 }