Skip to content
This repository has been archived by the owner on Feb 25, 2024. It is now read-only.

V1 release #14

Merged
merged 23 commits into from Jul 3, 2018
Merged

V1 release #14

merged 23 commits into from Jul 3, 2018

Conversation

ainformatico
Copy link
Owner

@ainformatico ainformatico commented Jun 10, 2018

Description

This is the V1 release:

  • Drop support for old Ruby versions
  • Update gems
  • Rubocop love
  • Add Coveralls support
  • Improve general coding
  • Add support for custom params (e.g private_domain)

TODO

expect(message.forwards_left).to eq(10)
expect(message.original).to be_a(Hash)
end
describe '#get' do

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Metrics/BlockLength: Block has too many lines. [31/25]

@ainformatico
Copy link
Owner Author

@mattrobbins changes on this PR are finished, just waiting for your changes and we can make the release of V1.

Support private domain and custom options
@@ -32,3 +33,5 @@ def register_uri(url, file, method = :get)
register_uri('http://api.mailinator.com:443/api/error?token=ABCD', 'error.response')
register_uri('http://api.mailinator.com:443/api/delete?msgid=abcd1234&token=ABCD', 'delete.response')
register_uri('http://api.mailinator.com:443/api/delete?msgid=1419696967-44152505-recipient&token=ABCD', 'delete.response')
register_uri('http://api.mailinator.com:443/api/inbox?private_domain=true&to=abcd1234&token=ABCD', 'inbox_private.response')
register_uri('http://api.mailinator.com:443/api/email?private_domain=true&msgid=1419696967-44152505-recipient&token=ABCD', 'email_private.response')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Metrics/LineLength: Line is too long. [148/80]

@@ -32,3 +33,5 @@ def register_uri(url, file, method = :get)
register_uri('http://api.mailinator.com:443/api/error?token=ABCD', 'error.response')
register_uri('http://api.mailinator.com:443/api/delete?msgid=abcd1234&token=ABCD', 'delete.response')
register_uri('http://api.mailinator.com:443/api/delete?msgid=1419696967-44152505-recipient&token=ABCD', 'delete.response')
register_uri('http://api.mailinator.com:443/api/inbox?private_domain=true&to=abcd1234&token=ABCD', 'inbox_private.response')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Metrics/LineLength: Line is too long. [124/80]

@@ -21,6 +20,8 @@ def register_uri(url, file, method = :get)
register_uri('https://api.mailinator.com/api/error?token=ABCD', 'error.response')
register_uri('https://api.mailinator.com/api/delete?msgid=abcd1234&token=ABCD', 'delete.response')
register_uri('https://api.mailinator.com/api/delete?msgid=1419696967-44152505-recipient&token=ABCD', 'delete.response')
register_uri('https://api.mailinator.com/api/inbox?private_domain=true&to=abcd1234&token=ABCD', 'inbox_private.response')
register_uri('https://api.mailinator.com/api/email?private_domain=true&msgid=1419696967-44152505-recipient&token=ABCD', 'email_private.response')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Metrics/LineLength: Line is too long. [145/80]

@@ -21,6 +20,8 @@ def register_uri(url, file, method = :get)
register_uri('https://api.mailinator.com/api/error?token=ABCD', 'error.response')
register_uri('https://api.mailinator.com/api/delete?msgid=abcd1234&token=ABCD', 'delete.response')
register_uri('https://api.mailinator.com/api/delete?msgid=1419696967-44152505-recipient&token=ABCD', 'delete.response')
register_uri('https://api.mailinator.com/api/inbox?private_domain=true&to=abcd1234&token=ABCD', 'inbox_private.response')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Metrics/LineLength: Line is too long. [121/80]

require 'spec_helper'

describe Mailinator::Inbox do
let(:inbox_private) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/BlockDelimiters: Avoid using {...} for multi-line blocks.

@@ -0,0 +1,37 @@
require 'spec_helper'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/FrozenStringLiteralComment: Missing magic comment # frozen_string_literal: true.

expect(message.subject).to eq('This is a Subject')
expect(message.request_id).to eq('638363')
expect(message.body).to include('<p>This is a body</p>')
expect(message.body_base_64).to include('This is a BASE64 body JVBERi0xLjUKJe')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Metrics/LineLength: Line is too long. [87/80]

expect(message.from_full).to eq('example@example.net')
expect(message.date).to be_a(DateTime)
expect(message.received)
.to include('from bmta1.example.com([9.9.9.9]) by mail.mailinator.com with SMTP (Postfix) for receipient@example.com; Fri, 11 May 2018 20:55:36 +0000 (UTC)')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Layout/MultilineMethodCallIndentation: Use 2 (not 0) spaces for indenting an expression spanning multiple lines.
Metrics/LineLength: Line is too long. [165/80]

expect(message.forwards_left).to eq(10)
expect(message.original).to be_a(Hash)
end
describe '#get' do

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Metrics/BlockLength: Block has too many lines. [29/25]

require 'date'

module Mailinator
module Models
class Email < Base
def transform_data
{
id: @data['data']['id'],
from_full: @data['data']['fromfull'],
date: DateTime.parse(@data['data']['headers']['date']),

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/DateTime: Prefer Date or Time over DateTime.

@ainformatico
Copy link
Owner Author

@mattrobbins I will now make the last changes to this PR, merge it and release V1. Thanks a lot for your contribution 😄

@ainformatico ainformatico merged commit d5a0969 into master Jul 3, 2018
@ainformatico ainformatico deleted the v1-release branch July 3, 2018 06:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants