Skip to content

Commit

Permalink
fix: unit test (all lower case) + docker build (npm install error)
Browse files Browse the repository at this point in the history
  • Loading branch information
kad-dirc committed Dec 17, 2024
1 parent a9bc108 commit d442ff7
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
1 change: 1 addition & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ FROM docker.io/node:lts-alpine3.17 AS build-component
RUN mkdir -p /usr/src/app
COPY ./viewer /usr/src/app
WORKDIR /usr/src/app
RUN npm config set registry http://registry.npmjs.org
RUN npm install
RUN npm run build

Expand Down
8 changes: 4 additions & 4 deletions config/config_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -81,28 +81,28 @@ func TestGeoSpatialCollections_Ordering(t *testing.T) {
name: "should return collections in default order (alphabetic)",
args: args{
configFile: "internal/engine/testdata/config_collections_order_alphabetic.yaml",
expectedOrder: []string{"A", "B", "C", "Z", "Z"},
expectedOrder: []string{"a", "b", "c", "z", "z"},
},
},
{
name: "should return collections in default order (alphabetic) - by title",
args: args{
configFile: "internal/engine/testdata/config_collections_order_alphabetic_titles.yaml",
expectedOrder: []string{"B", "C", "Z", "Z", "A"},
expectedOrder: []string{"b", "c", "z", "z", "a"},
},
},
{
name: "should return collections in default order (alphabetic) - extra test",
args: args{
configFile: "internal/engine/testdata/config_collections_unique.yaml",
expectedOrder: []string{"BarCollection", "FooCollection", "FooCollection"},
expectedOrder: []string{"bar_collection", "foo_collection", "foo_collection"},
},
},
{
name: "should return collections in explicit / literal order",
args: args{
configFile: "internal/engine/testdata/config_collections_order_literal.yaml",
expectedOrder: []string{"Z", "Z", "C", "A", "B"},
expectedOrder: []string{"z", "z", "c", "a", "b"},
},
},
{
Expand Down

0 comments on commit d442ff7

Please sign in to comment.