Guardian request limit exceeded

[robertcv]

Issue

TheGuardianAPI would crash if requests limit is exceeded. The exception was not informative because it's raised further in the request processing.

Description of changes

Now an informative exception is raised. Additionally, this helps with the falling test which is now skipped if this error is raised.

Includes

• Code changes
• Tests
• Documentation

[codecov-io]

Codecov Report

Merging #365 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #365      +/-   ##
==========================================
+ Coverage   85.21%   85.24%   +0.03%     
==========================================
  Files          33       33              
  Lines        1867     1871       +4     
  Branches      336      337       +1     
==========================================
+ Hits         1591     1595       +4     
  Misses        237      237              
  Partials       39       39

[PrimozGodec]

The code looks ok for me. Can you try to cover the case when API limit exceed with the unit-test if that is possible?

[ajdapretnar]

Codecov still not happy. Parts of code aren't covered with tests. @robertcv do you think that's something we should add or let it be?

[robertcv]

@ajdapretnar I add a test with mocked request data