Propagated HTTP errors in HTTPStore containsItem function

src/storage/httpStore.ts

@@ -76,6 +76,9 @@ export class HTTPStore<UrlRoot extends string | URL=string> implements AsyncStor
         // Just check headers if HEAD method supported
         const method = this.supportedMethods.has(HTTPMethod.HEAD) ? HTTPMethod.HEAD : HTTPMethod.GET;
         const value = await fetch(url, { ...this.fetchOptions, method });
+        if (value.status !== 200 && value.status !== 404) {
+            throw new HTTPError(String(value.status));
+        }
         return value.status === 200;
     }
 }

test/globalSetup.js

@@ -3,6 +3,9 @@ const express = require("express");
 
 const app = express();
 app.use(express.static("fixtures", { dotfiles: 'allow' }));
+app.use('/forbidden', function (_req, res) {
+    return res.status(403).end();
+});
 const server = app.listen(3000);
 
 module.exports = async () => {

test/storage/httpStore.test.ts

@@ -1,9 +1,10 @@
 (global as any).fetch = require('node-fetch');
 
+import { HTTPError } from "../../src/errors";
 import { HTTPStore } from "../../src/storage/httpStore";
 import { openArray } from "../../src/creation";
 
-describe("Test MemoryStore", () => {
+describe("Test HTTPStore", () => {
 
     const simpleFixtureStoreLE = new HTTPStore("http://localhost:3000/simple_LE.zarr");
     it("Can open simple fixture", async () => {
@@ -60,4 +61,9 @@ describe("Test MemoryStore", () => {
         const z = await openArray({ store: baseUrlStore, path: "simple_LE.zarr" });
         expect(z.shape).toEqual([8, 8]);
     });
+
+    const forbiddenStore = new HTTPStore("http://localhost:3000/forbidden");
+    it("Propagates HTTP server error", async () => {
+        await expect(openArray({ store: forbiddenStore })).rejects.toThrowError(HTTPError);
+    });
 });