@codecakes can you please assign this to me i am interested in working on this issue

@codecakes can you assign me this one

@codecakes can you please assign this to me i am interested in working on this issue

@Blacksujit feel free to raise the PR. it will be automatically assigned to whoever has a PR in review.

@codecakes can you assign me this one

@MAVRICK-1 feel free to raise the PR. it will be automatically assigned to whoever has a PR in review.

@codecakes was giving this a try but got stuck in pytests for this , any guide i could follow , i tested it on already implemented check for params not matching but it shows no error raised (i am on "main" branch )

@codecakes was giving this a try but got stuck in pytests for this , any guide i could follow , i tested it on already implemented check for params not matching but it shows no error raised (i am on "main" branch )

@RISHIKESHk07 share instructions on what steps you took to reproduce your issue including command, code and screenshot.

@codecakes wrote a new test file and tested it , on main branch

@codecakes wrote a new test file and tested it , on main branch

@RISHIKESHk07 Good effort to begin with but you are proving the point that you have already declared the method. You are not doing anything wrong but contradicting the issue this ticket asks to implement. The issue clearly states to check if the method, in your case method_with_params, is absent in either classes, then it should raise.

In your particular case, if you run this:

class BaseClass:
    def method_with_params(self, param1: int, param2: str):
        pass


class IncorrectImplementation(BaseClass, InterfaceProtocolCheckMixin):
    def method_with_params(self, param1: int):
        pass

IncorrectImplementation()

you will get a traceback like this because a.) you already have declared the method BUT b.) It has different signature.

Traceback (most recent call last):
    ...
    raise NotImplementedError(f"""Signature for {defined_method} not correct:
NotImplementedError: Signature for method_with_params not correct:
                Expected: ['self', 'param1', 'param2']
                Got: ['self', 'param1']

I suggest to read the description and goals again.

Additionally, when sharing a bug with a screenshot that you want others to try which has a reproducible code, share the code separately when sharing an issue on any ticket. I suggest to look at the following to understand what details to share as a helpful guide:

@codecakes Thanks for suggestions , i was not able get pytest working as shown in the terminal of photo ( figured it out after some trial and error ) . Will try to move forward in the issue

@codecakes Thanks for suggestions , i was not able get pytest working as shown in the terminal of photo ( figured it out after some trial and error ) . Will try to move forward in the issue

@RISHIKESHk07 There is nothing wrong with your pytest setup. It is working and that is why it fails.

See contributing to understand how to run tests generally.

@codecakes Thanks for suggestions , i was not able get pytest working as shown in the terminal of photo ( figured it out after some trial and error ) . Will try to move forward in the issue

@RISHIKESHk07 Check updated Implementation details section for clarity

@codecakes this is what i came up with , any suggestion i could make it better

@RISHIKESHk07 I am unable to understand the context properly.
I suggest you follow the guidelines provided here when sharing an issue. See here and If you get stuck to understand how to share an issue and your findings. See what happens when standards are not followed. This is what I see when I click on your screeshot:

From whatever I can gather from your screenshot, once you have implemented the code and written a unit test for it, raise a PR

@codecakes this is what i came up with , any suggestion i could make it better

Is this issue still open?

hello, could u pls assign this issue to me i would like to work on it.

@codecakes hello, I'd like to work on this issue too.
I have implemented and manually tested the functionality needed, just need to implement auto testing with pytest.