The error messages have been:
System Error o59if2646136wek.132
System Error v4if2628789wea.79
Sy...

It looks like these errors are inherent in communicating with IMAP,
and there's often no reasona...

For whatever reason, when Travis switched to Ruby 1.9.3-p194 on their working
machines, the test...

I've no idea why my previous commit broke the build.

* Link to /conversations in the page header. I want to be able to
navigate to conversations ea...

In order to do this, we need to be able to load individual conversations, and link to them in th...

It always felt a bit wrong that the `ActiveRecord` implementation behind the conversation index ...

`Conversation` instances don't store references to messages using their unique
IDs, but do store...

The view implementation here is trivial and wrong, but this should demonstrate the skeleton of h...

Delete data rather than resetting elastic search index between tests.

By using the same index for all tests, only deleting the data in the index between each test, we...

Yeah, I knew it was one word. Who said I didn't...

Explain that you'll need to install elasticsearch to use Sauron.

This reverts commit 2d7d0cb10faa6276029b15c4358f4704a7a03667.

Apparently @tomafro has tried thi...

Issuing a `refresh` command seems to ensure the index is ready for
searching. Hopefully it will ...

The custom foreman export upstart template ensures that sauron is
started at boot time and stopp...

Ideally this would prepend the first two lines (start & stop commands)
to the default template, ...

Somehow, in a fit of stashing and popping, the actual assertion in the
acceptance test disappear...

This lets us quickly see who was involved in a conversation.

It's likely that this presentation...

I've deliberately given this an obnoxious name.

This will be useful when re-importing.

This hits both the MessageRepository and the ConversationRepository in
order to check that both ...

This change reduces the time taken from ~30 seconds to ~15 seconds.

It's also probably better t...

We chose the name `logged_in` rather than `login` because the latter is
an imperative which impl...

Making message_stub do everything that stub_message used to do allows us
to remove stub_message....

This is because if a stub receives an unexpected method call, or an
expectation on a stub isn't ...

We think it's better to be explicit about the interface our stub
implements.

We want our tests to fail fast if we unintentionally stub and call a
method that doesn't exist. ...

Also ensure valid HTML by not adding empty <ul> elements.

It felt odd that searching with an empty query displayed a search
results page with no matching ...

We've chosen to implement the microformat datetime design pattern[1]
which makes it easier to wr...

Although the default order may have been determined by "closest match",
this wasn't obvious from...

Patch ActiveSupport::Multibyte::Chars so that as_json works

This is just based on which attributes are used in `WHERE` queries.
Perhaps I'm being a bit naïv...

This probably won't be useful for new messages, but might be of use for
older ones. I'm not plan...

It's possible that the importer will try to add the same `Message` to the `ConversationRepositor...

I believe this test is really concerned with hinting to the mailbox that the importer doesn't wa...

This is required for the `MailImporter` to interact with the
`ConversationRepository` properly.

- This is part of a larger story to present conversations rather than
individual messages. We ...

Not all messages have To headers, so ParsedMail ensures we always return something, even when no...

This reverts commit aaf1ff540ba932d6d1ee26f81627fc2eb7a5cbb6.

Adding the --trace flag to the ra...

Including the --trace switch should help us debug some of the errors we
see every so often.

If an error is raised attempted to decode them, force them to UTF-8, replacing any invalid bytes.

We've hidden emails from twitter, pivotal tracker and google calendar from the main view, but th...

It proved very difficult to write a test to reproduce the problem with
this message [1]. Given t...

We noticed on the production server that search for cron, for example, would return messages whe...

All messages imported after this commit is deployed should be searchable by their bodies. We pl...

ParsedMail seems a more descriptive name for the parsed version of a raw email.

For the most part we rely on the mail gem to parse our raw emails. However, it has a number of ...

I think the default rake task already calls db:test:prepare; all we need to do is get the 'devel...

This means we don't have to keep every constant referred to in any
migration around in the codeb...

The Mail gem parses multiple `Delivered-To` headers into a wacky Array
of 'Fields', but we were ...

We've switched to ElasticSearch.

This has the potential to be more informative in the future.

Once a certain number of messages have been imported, an http connection limit within the OS is ...

By performing the migration in batches and reinitializing the index each time, I hope to avoid a...

Wip elastic search into shape

Using Elastic Search rather than ActiveRecord to index our messages enable us to easily perform ...

mail["Delivered-To"] returns a Mail::OptionalField, while mail["Delivered-To"].to_s returns an A...

* Using the `UID SEARCH` IMAP command in conjunction with the
`X-GM-LABELS` GMail extension at...

The hash can be seen as an identifier of the message as a whole (as opposed to a single email). ...

When I added this test, it was to ensure that the `has_many` was
correctly configured. However t...

The names of the `received` and `not-received` CSS classes are a bit
misleading, but we can addr...

No need to order by ID since there should now only ever be a single
message index record for a g...