Commit203da23

committed

Add a few FIXMEs re: better use of xfail

1 parentfc02230 commit203da23Copy full SHA for 203da23

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -100,6 +100,7 @@ def test_includes_order(self):`
`100`	`100`	`# values must be considered as soon as they get them`
`101`	`101`	`assertr_config.get_value("diff","tool")=="meld"`
`102`	`102`	`try:`
	`103`	`+# FIXME: Split this assertion out somehow and mark it xfail (or fix it).`
`103`	`104`	`assertr_config.get_value("sec","var1")=="value1_main"`
`104`	`105`	`exceptAssertionErrorase:`
`105`	`106`	`raiseSkipTest("Known failure -- included values are not in effect right away")frome`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -85,6 +85,7 @@ def setup(self):`
`85`	`85`	`"array": [42],`
`86`	`86`	`}`
`87`	`87`
	`88`	`+# FIXME: Mark only the /proc-prefixing cases xfail, somehow (or fix them).`
`88`	`89`	`@pytest.mark.xfail(`
`89`	`90`	`reason="Many return paths prefixed /proc/cygdrive instead.",`
`90`	`91`	`raises=AssertionError,`
`@@ -103,7 +104,7 @@ def test_cygpath_ok(self, case):`
`103`	`104`	`@skipUnless(sys.platform=="cygwin","Paths specifically for Cygwin.")`
`104`	`105`	`@ddt.data(`
`105`	`106`	`(r"./bar","bar"),`
`106`		`- (r".\bar","bar"),`
	`107`	`+ (r".\bar","bar"),# FIXME: Mark only this one xfail, somehow (or fix it).`
`107`	`108`	`(r"../bar","../bar"),`
`108`	`109`	`(r"..\bar","../bar"),`
`109`	`110`	`(r"../bar/.\foo/../chu","../bar/chu"),`

Comments

(0)