From 41380593f7c8509f9da210b5f0a594edf9cc2a32 Mon Sep 17 00:00:00 2001
From: MarLatte <86508134+marlatte@users.noreply.github.com>
Date: Sat, 20 May 2023 02:37:41 -0400
Subject: [PATCH] Update palindromes to handle numbers better

The previous solution removed numbers entirely, whereas this one treats
them like letters and checks if they are evenly spaced.

More importantly, the old solution test seemed to check if the numbers
were palindromic, but because the solution replaced them with "", it
wasn't testing what it seemed to.

I added a new test to differentiate between the palindromic "rac3e3car"
and the non-palindromic "r3ace3car".
---
 09_palindromes/solution/palindromes-solution.js      | 2 +-
 09_palindromes/solution/palindromes-solution.spec.js | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/09_palindromes/solution/palindromes-solution.js b/09_palindromes/solution/palindromes-solution.js
index bc5ef94..7950015 100644
--- a/09_palindromes/solution/palindromes-solution.js
+++ b/09_palindromes/solution/palindromes-solution.js
@@ -1,5 +1,5 @@
 const palindromes = function (string) {
-  const processedString = string.toLowerCase().replace(/[^a-z]/g, "");
+  const processedString = string.toLowerCase().replace(/[^a-z0-9]/g, "");
   return processedString.split("").reverse().join("") == processedString;
 };
 
diff --git a/09_palindromes/solution/palindromes-solution.spec.js b/09_palindromes/solution/palindromes-solution.spec.js
index e8d7e6a..6e31d91 100644
--- a/09_palindromes/solution/palindromes-solution.spec.js
+++ b/09_palindromes/solution/palindromes-solution.spec.js
@@ -24,4 +24,7 @@ describe('palindromes', () => {
   test('works with numbers in a string', () => {
     expect(palindromes('rac3e3car')).toBe(true);
   });
+  test('works with unevenly spaced numbers in a string', () => {
+    expect(palindromes('r3ace3car')).toBe(false);
+  });
 });