Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow stubbing without using IDs #5

Open
boothj5 opened this issue Aug 26, 2018 · 2 comments
Open

Allow stubbing without using IDs #5

boothj5 opened this issue Aug 26, 2018 · 2 comments

Comments

@boothj5
Copy link
Collaborator

boothj5 commented Aug 26, 2018

Currently the stbbr_for_id functionality allows stubbing Stanza responses based on the Stanza ID.

Clients generally use random numbers for IDs, so Stabber should never be reliant on them.

A better approach would be to stub a response based on the stanza contents.

jubalh added a commit to jubalh/profanity that referenced this issue Aug 30, 2018
Functional tests were commented out because of:
profanity-im#1010
An issue was raised for stabber:
profanity-im/stabber#5
Once this issue is resolved functional tests should be enabled again.
jubalh added a commit to jubalh/profanity that referenced this issue Aug 30, 2018
Functional tests were commented out because of:
profanity-im#1010
An issue was raised for stabber:
profanity-im/stabber#5
Once this issue is resolved functional tests should be enabled again.
@jubalh
Copy link
Member

jubalh commented Aug 30, 2018

Discovered through profanity-im/profanity#1010

profanity-im/profanity@171b6e7 disables functional tests in profanity.

Once this issue is resolved a PR to re-enable them should be made or an issue created.

@jubalh
Copy link
Member

jubalh commented Feb 19, 2019

@boothj5 I might take a look at this soon. What exactly is an example for 'stanza content' we should use?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants