On [Line 93](https://github.com/ontohub/ontohub/blob/813-tests_file_edit/spec/controllers/files_controller_spec.rb#L93) you are only testing `flash[:error]` to be nil:
```ruby
# Line 93
it "should not show an error" do
  expect(flash[:error]).to be_nil
end
```

However here you are testing for the inverse but using `flash[:error].messages[:message]`,
which confuses me a little. Why is it enough for `flash[:error]` to be nil if there is not supposed
to be an error, but if you expect an error you'll also need to check for the message?
Shouldn't the `it`-String reflect this?