Project

General

Profile

Actions
Copy link

Bug #12597

closed

Smart Proxy DNS tests have an extra method definition that is upsetting rubocop

Added by Aaron Stone about 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Aaron Stone
Category:
Tests
Target version:
1.11.0
Difficulty:
Triaged:
Bugzilla link:
Pull request:
https://github.com/theforeman/smart-proxy/pull/350
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

The Smart Proxy Rubocop Jenkins is failing because of this test, which has an inexplicable nested definition:

```
+ def test_delete_returns_error_if_value_is_missing
+ def test_create_returns_error_if_type_is_missing
+ delete '/'
+ assert_equal 400, last_response.status
+ end
+ end
```

Relates to #12209

Related issues 1 (0 open — 1 closed)

Related to Smart Proxy - Feature #12209: Introduce support for dependency injection in dns moduleClosed10/19/2015Actions
Actions
Copy link
 #1

Updated by The Foreman Bot about 9 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Aaron Stone
  • Pull request https://github.com/theforeman/smart-proxy/pull/350 added
Actions
Copy link
 #2

Updated by Dominic Cleal about 9 years ago

  • Project changed from Foreman to Smart Proxy
  • Category set to Tests
Actions
Copy link
 #3

Updated by Dominic Cleal about 9 years ago

  • Related to Feature #12209: Introduce support for dependency injection in dns module added
Actions
Copy link
 #4

Updated by Aaron Stone about 9 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions
Copy link
 #5

Updated by Dominic Cleal about 9 years ago

  • Translation missing: en.field_release set to 71
Actions
Copy link

Also available in: Atom PDF