Skip to content

style: fix PEP 8 violations in exercise test files#362

Open
AleksandrShushunov wants to merge 1 commit into
hexlet-basics:mainfrom
AleksandrShushunov:refactor/pep8-and-docstrings
Open

style: fix PEP 8 violations in exercise test files#362
AleksandrShushunov wants to merge 1 commit into
hexlet-basics:mainfrom
AleksandrShushunov:refactor/pep8-and-docstrings

Conversation

@AleksandrShushunov
Copy link
Copy Markdown

style: fix PEP 8 violations in exercise test files

Устранены предупреждения pycodestyle по умолчанию (лимит 79 символов в строке) в шести тестовых файлах упражнений. Логика тестов, проверки и ожидаемые значения не изменялись — затронуты только пробелы и переносы строк.

Изменения:

  • modules/40-define-functions/150-return/test_solution.py Добавлена вторая пустая строка между import и определением def test() (E302: expected 2 blank lines, found 1).

  • modules/45-logic/15-bool-strings/test_solution.py Добавлена вторая пустая строка между import и определением def test1() (E302: expected 2 blank lines, found 1).

  • modules/48-conditionals/40-if-else/test_solution.py Три длинных assert обёрнуты в круглые скобки, чтобы строки с URL-литералами укладывались в 79 символов (E501: line too long, 83, 87 и 80 символов). Это согласуется со стилем уже присутствующих ниже многострочных assert.

  • modules/50-loops/80-for-in-range/test_solution.py Длинный assert с fizzbuzz(15) обёрнут в круглые скобки, чтобы уложиться в 79 символов (E501: line too long, 86 символов). Ожидаемая выходная строка не изменена.

  • modules/40-define-functions/250-named-arguments/test_solution.py Длинный assert с trim_and_repeat обёрнут в круглые скобки (E501: line too long, 80 символов).

  • modules/50-loops/23-aggregation-strings/test_solution.py Длинный assert с sanitize_phone_number обёрнут в круглые скобки (E501: line too long, 81 символ).

Closes #360

style: fix PEP 8 violations in exercise test files

Устранены предупреждения pycodestyle по умолчанию (лимит 79 символов
в строке) в шести тестовых файлах упражнений. Логика тестов, проверки
и ожидаемые значения не изменялись — затронуты только пробелы и
переносы строк.

Изменения:

- modules/40-define-functions/150-return/test_solution.py
  Добавлена вторая пустая строка между import и определением
  def test() (E302: expected 2 blank lines, found 1).

- modules/45-logic/15-bool-strings/test_solution.py
  Добавлена вторая пустая строка между import и определением
  def test1() (E302: expected 2 blank lines, found 1).

- modules/48-conditionals/40-if-else/test_solution.py
  Три длинных assert обёрнуты в круглые скобки, чтобы строки
  с URL-литералами укладывались в 79 символов (E501: line too
  long, 83, 87 и 80 символов). Это согласуется со стилем уже
  присутствующих ниже многострочных assert.

- modules/50-loops/80-for-in-range/test_solution.py
  Длинный assert с fizzbuzz(15) обёрнут в круглые скобки,
  чтобы уложиться в 79 символов (E501: line too long, 86
  символов). Ожидаемая выходная строка не изменена.

- modules/40-define-functions/250-named-arguments/test_solution.py
  Длинный assert с trim_and_repeat обёрнут в круглые скобки
  (E501: line too long, 80 символов).

- modules/50-loops/23-aggregation-strings/test_solution.py
  Длинный assert с sanitize_phone_number обёрнут в круглые
  скобки (E501: line too long, 81 символ).

Closes hexlet-basics#360
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PEP8 WIP

1 participant