On Fri, Feb 29, 2008 at 12:29 AM, Erich Hoover ehoover@mines.edu wrote:
If a test fails, it has to be wrapped in todo_wine, else make test will fail.
On Thu, Feb 28, 2008 at 11:44 PM, James Hawkins truiken@gmail.com wrote:
I would argue that make test should fail for regressions, though I don't know when exactly this particular problem surfaced. Re-ording the tests can actually mask the problem as well as inserting todo_wine, which would be preferred?
Erich Hoover ehoover@mines.edu
On Fri, Feb 29, 2008 at 1:17 AM, Erich Hoover ehoover@mines.edu wrote:
Um make test does fail for regressions. make test fails if an ok call fails and is not covered by todo_wine, and it also fails when a test in a todo_wine block succeeds. That's how we keep track of the tests. If reordering the tests masks the problem, what's the point of the tests?
On Fri, Feb 29, 2008 at 1:18 AM, James Hawkins truiken@gmail.com wrote:
The tests are for checking the sharing permissions problem, which causes WritePrivateProfileString to fail when the profile file is opened by CreateFile before WritePrivateProfileString is called. The tests failing now is a result of a problem with wine caching the profile information from when the call previously succeeded (the cache is not updated properly when the file is deleted). What I'm trying to say is that the tests are currently exposing a separate issue, so a separate set of tests would probably be appropriate - problems with caching are outside the scope of the patches I submitted.
Erich Hoover ehoover@mines.edu
-
Erich Hoover -
James Hawkins