Fix integ spec and ordering issue

smithy-lang · Aug 23, 2023 · 8c6a1ea · 8c6a1ea
1 parent 350efa6
commit 8c6a1ea
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 5 deletions.
diff --git a/codegen/projections/high_score_service/lib/high_score_service/auth.rb b/codegen/projections/high_score_service/lib/high_score_service/auth.rb
diff --git a/codegen/projections/white_label/lib/white_label/auth.rb b/codegen/projections/white_label/lib/white_label/auth.rb
diff --git a/codegen/projections/white_label/spec/auth_spec.rb b/codegen/projections/white_label/spec/auth_spec.rb
diff --git a/codegen/smithy-ruby-codegen-test/integration-specs/auth_spec.rb b/codegen/smithy-ruby-codegen-test/integration-specs/auth_spec.rb
@@ -270,6 +270,8 @@ module Auth
           expect(resolved).to eq(identity)
           resolved
         end
+        # temporarily disabled because not implemented
+        expect_any_instance_of(Hearth::Signers::HTTPDigest).to receive(:sign)
         client.http_digest_auth({})
       end
     end

diff --git a/...n/src/main/java/software/amazon/smithy/ruby/codegen/generators/AuthResolverGenerator.java b/...n/src/main/java/software/amazon/smithy/ruby/codegen/generators/AuthResolverGenerator.java
@@ -15,6 +15,7 @@
 
 package software.amazon.smithy.ruby.codegen.generators;
 
+import java.util.Comparator;
 import java.util.HashSet;
 import java.util.Map;
 import java.util.Optional;
@@ -103,9 +104,9 @@ private void renderAuthSchemesConstant(RubyCodeWriter writer) {
         writer
                 .openBlock("SCHEMES = [")
                 .call(() -> {
-                    authTraitsSet.stream().forEach(trait -> {
-                        renderAuthScheme(writer, trait);
-                    });
+                    authTraitsSet.stream()
+                            .sorted(Comparator.comparing(Trait::toShapeId))
+                            .forEach(trait -> renderAuthScheme(writer, trait));
                 })
                 .unwrite(",\n")
                 .write("")