twisted / twisted/twisted

trial path/to/package// fails immediately with AssertionError

Open
#2,823 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug new priority-normal trial
Dominant language
Python
Stars
6k
Forks
1.2k
Avg merge
2d 10h
Merged PRs (30d)
10

Description

exarkun's avatar @exarkun reported
Trac ID trac#2823
Type defect
Created 2007-09-06 22:28:12Z

The double trailing slash causes trial to find a badly named __init__ because of the behavior of filenameToModuleName which results in weird state which triggers an assertion from twisted.python.modules.

Probably trial shouldn't use filenameToModuleName at all, since that function sucks (#2822).

Searchable metadata
trac-id__2823 2823
type__defect defect
reporter__exarkun exarkun
priority__normal normal
milestone__ 
branch__ 
branch_author__ 
status__new new
resolution__None None
component__trial trial
keywords__ 
time__1189117692000000 1189117692000000
changetime__1330080692000000 1330080692000000
version__None None
owner__ 

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing trial path/to/package// and trace how trial calls filenameToModuleName. Read the related behavior in twisted.python.modules, then inspect issue #2822 for context on that function. Done means the double trailing slash no longer reaches the assertion and trial reports or handles the path correctly.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.