Andrew hung nguyen - is_palindrome() fixes and method naming convention changes (#75)

* Adding a gitignore to avoid adding pyc

* Separated string, statistics, and bitwise operations to separate files and created unit tests

* is_palindrome and string manipulation test method names modified

* is_palindrome and string manipulation test method names modified

Author: AndrewHUNGNguyen

easyPythonpi/__pycache__/easyPythonpi.cpython-311.pyc

@@ -9,7 +9,6 @@ def remove_punctuation(my_str:'str')->'str':
     for char in my_str:
         if char not in punctuations:
             no_punct = no_punct + char
-    #return no_punct
     return no_punct
 
 def count_vowels(ip_str:'str')->'dict':
@@ -29,12 +28,23 @@ def count_vowels(ip_str:'str')->'dict':
     #return the count dictionary
     return count
 
-def ispalindrome(x:'str')->'bool':        # To check if the given parameter is palindrome or not
-    x=str(x)  #explicitly convert into string data type so as to iterate through each character
+# To check if the given parameter is palindrome or not
+def is_palindrome(x:'str')->'bool':        
+    # convert into string data type so as to iterate through each character
+    x=str(x)  
+
+    # remove whitespace, lower case letters, and then remove punctuations
+    x = x.replace(" ", "")
+    x = x.lower()
+    x = remove_punctuation(x)
+
+    # Convert original string to its reversed version
     r=''
     for i in range(len(x)-1,-1,-1):
-                r=r+x[i]
-    if x==r:    # if the parameter get matched with its reverse then returns true othewise false
-            return True
+        r=r+x[i]
+
+    # if the parameter get matched with its reverse then returns true othewise false
+    if x==r:    
+        return True
     else:
-            return False 
+        return False 

easyPythonpi/methods/stringmainpulation.py

@@ -3,66 +3,70 @@
 # Import the function you want to test
 from methods.stringmainpulation import *
 
-class TestCountVowels(unittest.TestCase):
-    def test_no_vowels(self):
+class TestStringManipulation(unittest.TestCase):
+    def test_count_vowels_string_no_vowels(self):
         result = count_vowels("rhythm")
         self.assertEqual(result, {'a': 0, 'e': 0, 'i': 0, 'o': 0, 'u': 0})
 
-    def test_all_vowels(self):
+    def test_count_vowels_string_has_vowels(self):
         result = count_vowels("aeiouAEIOU")
         self.assertEqual(result, {'a': 2, 'e': 2, 'i': 2, 'o': 2, 'u': 2})
 
-    def test_mixed_string(self):
+    def test_count_vowels_string_vowels_and_consonants(self):
         result = count_vowels("Hello, World!")
         self.assertEqual(result, {'a': 0, 'e': 1, 'i': 0, 'o': 2, 'u': 0})
 
-    def test_empty_string_counting_vowels(self):
+    def test_count_vowels_empty_string(self):
         result = count_vowels("")
         self.assertEqual(result, {'a': 0, 'e': 0, 'i': 0, 'o': 0, 'u': 0})
 
-    def test_case_insensitive(self):
+    def test_count_vowels_empty_string(self):
         result = count_vowels("Python Is Awesome")
         self.assertEqual(result, {'a': 1, 'e': 2, 'i': 1, 'o': 2, 'u': 0})
 
-    def test_empty_string(self):
-        result = ispalindrome("")
+    def test_count_vowels_string_non_ascii_chars_only(self):
+        result = count_vowels("#!@$% ?")
+        self.assertEqual(result, {'a': 0, 'e': 0, 'i': 0, 'o': 0, 'u': 0})
+
+    def test_is_palindrome_empty_string(self):
+        result = is_palindrome("")
         self.assertTrue(result)
 
-    def test_single_character(self):
-        result = ispalindrome("a")
+    def test_is_palindrome_whitespace_only(self):
+        result = is_palindrome(" ")
         self.assertTrue(result)
 
-    def test_palindrome_with_spaces(self):
-        result = ispalindrome("A man a plan a canal Panama")
+    def test_is_palindrome_single_character(self):
+        result = is_palindrome("a")
         self.assertTrue(result)
 
-    def test_palindrome_with_punctuation(self):
-        result = ispalindrome("Was it a car or a cat I saw?")
+    def test_is_palindrome_sentence_without_punctuation(self):
+        result = is_palindrome("A man a plan a canal Panama")
         self.assertTrue(result)
 
-    def test_non_palindrome(self):
-        result = ispalindrome("Hello, World!")
+    def test_is_palindrome_sentence_with_punctuation(self):
+        result = is_palindrome("Was it a car or a cat I saw?")
+        self.assertTrue(result)
+
+    def test_is_palindrome_sentence_not_palindrome(self):
+        result = is_palindrome("Hello, World!")
         self.assertFalse(result)    
 
-    def test_empty_string_remove_punctuation(self):
+    def test_remove_punctuation_empty_string_remove_punctuation(self):
         result = remove_punctuation("")
         self.assertEqual(result, "")
 
-    def test_no_punctuation(self):
+    def test_remove_punctuation_sentence_no_punctuation(self):
         result = remove_punctuation("This is a test string")
         self.assertEqual(result, "This is a test string")
 
-    def test_with_punctuation(self):
+    def test_remove_punctuation_sentence_with_punctuation(self):
         result = remove_punctuation("Hello, World! This is a test.")
         self.assertEqual(result, "Hello World This is a test")
 
-    def test_mixed_characters(self):
+    def test_remove_punctuation_mixed_characters(self):
         result = remove_punctuation("123@abc#def%")
         self.assertEqual(result, "123abcdef")
 
-    def test_unicode_punctuation(self):
-        result = remove_punctuation("Unicode — string: ™ ©")
-        self.assertEqual(result, "Unicode  string ")
-
 if __name__ == '__main__':
     unittest.main()