Fix test and warnings

4 files changed, 11 insertions, 28 deletions

diff --git a/CHANGELOG.md b/CHANGELOG.md
index e74c531..a756cc7 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,7 +7,7 @@
   misused, resulting in poor code.
 - `list.LengthMismatch` has been removed.
 - The mistakenly public `bit_array.do_inspect` function has been removed.
-- Fixed the `dynamic.classification` function for boolean values.
+- Fixed the `dynamic.classification` function for bools.
 
 ## v0.36.0 - 2024-02-26
 
diff --git a/src/gleam_stdlib.mjs b/src/gleam_stdlib.mjs
index 821782d..40fcc3f 100644
--- a/src/gleam_stdlib.mjs
+++ b/src/gleam_stdlib.mjs
@@ -498,6 +498,8 @@ export function decode64(sBase64) {
 export function classify_dynamic(data) {
   if (typeof data === "string") {
     return "String";
+  } else if (typeof data === "boolean") {
+    return "Bool";
   } else if (data instanceof Result) {
     return "Result";
   } else if (data instanceof List) {
diff --git a/test/gleam/dynamic_test.gleam b/test/gleam/dynamic_test.gleam
index 5f22c9e..4b6c797 100644
--- a/test/gleam/dynamic_test.gleam
+++ b/test/gleam/dynamic_test.gleam
@@ -1,5 +1,5 @@
-import gleam/dynamic.{DecodeError}
 import gleam/dict
+import gleam/dynamic.{DecodeError}
 import gleam/option.{None, Some}
 import gleam/result
 import gleam/should
@@ -1505,10 +1505,6 @@ pub fn decode9_test() {
   )
 }
 
-type ClassifyAtom {
-  ClassifyAtom
-}
-
 pub fn classify_test() {
   dynamic.from(True)
   |> dynamic.classify
diff --git a/test/gleam/string_test.gleam b/test/gleam/string_test.gleam
index bb3f502..684bc1b 100644
--- a/test/gleam/string_test.gleam
+++ b/test/gleam/string_test.gleam
@@ -1,5 +1,5 @@
-import gleam/option.{None, Some}
 import gleam/dict
+import gleam/option.{None, Some}
 import gleam/order
 import gleam/should
 import gleam/string
@@ -846,31 +846,16 @@ pub fn inspect_test() {
   string.inspect(fn() { Nil })
   |> should.equal("//fn() { ... }")
 
-  string.inspect(fn(a) {
-    a
-    Nil
-  })
+  string.inspect(fn(_) { Nil })
   |> should.equal("//fn(a) { ... }")
 
-  string.inspect(fn(a, b) {
-    a
-    b
-    Nil
-  })
+  string.inspect(fn(_, _) { Nil })
   |> should.equal("//fn(a, b) { ... }")
 
-  string.inspect(fn(x, y) {
-    x
-    y
-    Nil
-  })
+  string.inspect(fn(_, _) { Nil })
   |> should.equal("//fn(a, b) { ... }")
 
-  string.inspect(fn(foo: Int, bar: String) -> Bool {
-    foo
-    bar
-    False
-  })
+  string.inspect(fn(_: Int, _: String) -> Bool { False })
   |> should.equal("//fn(a, b) { ... }")
 
   string.inspect(#(InspectTypeOne, InspectTypeTwo))
@@ -916,9 +901,9 @@ pub fn target_inspect_test() {
 }
 
 @target(erlang)
-import gleam/regex
-@target(erlang)
 import gleam/dynamic.{type Dynamic}
+@target(erlang)
+import gleam/regex
 
 // Test inspect on Erlang atoms valid and invalid in Gleam