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

Refactored and reformatted SmtpSampler #259

Closed
wants to merge 1 commit into from
Closed

Conversation

@ham1
Copy link
Contributor

ham1 commented Jan 27, 2017

  • Makes sample method much easier to read.
  • Minor changes to SendMailCommand

I understand this is a fairly big change with no tests but I thought I'd submit anyway in the hope that someone had some good SMTP tests or someone was willing to review or even just apply a subset of the changes.

…ier to read.

Minor changes to SendMailCommand
asfgit pushed a commit that referenced this pull request Jan 30, 2017
 Part 1/2 of PR contributed by Graham Russell
 comment #259

git-svn-id: https://svn.apache.org/repos/asf/jmeter/trunk@1780960 13f79535-47bb-0310-9956-ffa450edef68
@asfgit asfgit closed this in aa7d5db Jan 30, 2017
@ham1 ham1 deleted the ham1:smtp-refactor branch Jan 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.