It's really clever

woooo

return instead of print. Read this for more info

Approved

python new test framework is required. updated in this fork

unnecessary variable

you could just directly return int(s), it gives the same effect

niether is string2

Why is all this?

string2 is nothing but len(ending), and should consider naming those variables correctly they should not be strings.

Ruby 3.0 should be enabled, read this to learn how to do it

Please organize the structure of test fixture as following (although it has been mentioned in the attached link, I'm repeated here again)

describe "<This message should describe the categories of test groups inside this block>" do
  it "<A short message describing this test group>" do
    expect(...).to eq(...) #Assertions
    #Or Test.assert_equals(user_response, reference_response)
  end
end

string1 is not a string

At least the result should be a bool.

Boolean.