Anotated 'make test' log

Max TenEyck Woodbury max at mtew.isa-geek.net
Sat Jun 12 19:26:13 CDT 2010


I'm trying to get a start on helping with wine development. I'm still in 
the process of reading the developer's guide, and I think I am making 
progress on that. As a test of my understanding, I started to annotate 
the output of 'make -k test'. I'd like your opinion on the usefulness of 
this. A few reservations:
* I started from the top of the log. If I continue, I will be selecting
   sections that I find interesting.
* I am not going to produce patches until someone says that the
   annotations are going in the right direction. This will change when I
   become more familiar with the community's opinion of my efforts.

../../../tools/runtest -q -P wine -M advapi32.dll -T ../../.. -p
   advapi32_test.exe.so cred.c && touch cred.ok

  Note: This test is expected to produce 2 error reports

  call is pCredReadA(TEST_TARGET_NAME, -1, 0, &cred)
   implementation at dll/advavi32/cred.c - CredReadA
   parameter names: TargetName, Type, Flags, *Credentials
   conversion to CreadReadW
   implementation at same - CredReadW
fixme:cred:CredReadW unhandled type -1
  Fixup:
   Convert 'if (Type != CRED_TYPE_GENERIC && Type != 
CRED_TYPE_DOMAIN_PASSWORD)'
   to a switch(case?) with all known types enumerated. Put unimplemented 
types
   at the end with a 'fixup' then fall through to the default case which 
sets
   the error code and fails.
  Note: In other words, don't request fixup for improper type codes.

  call is pCredReadA(TEST_TARGET_NAME, CRED_TYPE_GENERIC, 0xdeadbeef, &cred)
   see above for details
fixme:cred:CredReadW unhandled flags 0xdeadbeef
  Fixup:
   Get a list of all known flags. Build a mask for all of them. Check 
for any
   unknown flags set and issue a warning if there are any. Build a mask 
of all
   implemented flags (currently none) and issue a fixme if any of those are
   set.
  Note: In other words, don't request fixup for undefined flags set.
  Question:
   Should this be an failure exit? Alternatively, should there be a list of
   flags that, if set trigger a failure exit?
  Note: Only fail if the flags indicate an option that can not be ignored.

  Note: No sucess tests.
  Note: Possible memory leak of returned credentials buffers?

  call is pCredWriteA(&new_cred, 0)
   implementation at same - CredWriteA
   parameter names: Credential, Flags
   new_cred.UserName = 'winetest'
   new_cred.TargetName = TEST_TARGET_NAME
   conversion to CredWriteW
   implementation at same - CredWriteW
err:cred:CredWriteW bad username L"winetest"
  Note: This is an expected error.
  Note: Should it mention 'syntax' since it is checking for \ and @?

  Note: No bad flags test - Fixup per CredReadW above.
  Note: No bad type test - Fixup per CredReadW above.

  call is pCredWriteA(&new_cred, 0)
   see above for details.
   new_cred.UserName = NULL
err:cred:CredWriteW bad username (null)
  Note: This is an expected error.

  Note: No sucess tests.
  Note: Possible memory leak of returned credentials buffers?

  call is pCredDeleteA(TEST_TARGET_NAME, -1, 0)
   implementation at same - CredDeleteA
   parameter names:  TargetName, Type, Flags
   conversion to CredDeleteW
fixme:cred:CredDeleteW unhandled type -1
   Fixup: see CredReadA above

  call is pCredDeleteA(TEST_TARGET_NAME, CRED_TYPE_GENERIC, 0xdeadbeef)
   se above for details.
fixme:cred:CredDeleteW unhandled flags 0xdeadbeef
  fixup: see CreadReadA above

  Note: No sucess tests.

  Note: Test order is questionable. A better order would be Write, Read then
   Delete with tests that are expected to succeed added.

  call is pCredReadDomainCredentialsA(&info, 0, &count, &creds
   implementation at same - CredReadDomainCredentialsA
   parameter names: TargetInformation, Flags, *Size, **Credentials
   conversion to CredReadDomainCredentialsW
fixme:cred:CredReadDomainCredentialsW (0x15ddf8, 0x0, 0x33fd18, 
0x33fc4c) stub
  Fixup: No suggestion




More information about the wine-devel mailing list