====== Bug Triage Day October 25-26th 2008 ======

===== Who's running it? =====
(on auto-pilot)

===== When =====
October 25th, 26th in #pear-bugs on irc.efnet.org

See [[http://www.google.com/calendar/embed?src=rslkppoio4rtq6nm7b6fptclfo%40group.calendar.google.com]]

===== Attendees =====
  * cweiske
  * doconnor
  * kguest
  * amir
  * justinpatrin
  * till
  * avb
  * Tatsuya Tsuruoka


===== What are we doing =====
  * Followup on [[http://pear.php.net/bugs/search.php?cmd=display&search_for=PHPCS&php_os=&boolean=0&bug_type=&bug_age=0&bug_updated=0&order_by=ts1&direction=DESC&packagever=&phpver=&limit=30&handle=&assign=&maintain=&showmenu=1&showmenu=1&status=Open|PHPCS patches]]
    * <del>Validate</del> (amir)
    * <del>HTTP_Session2</del> (till)
    * <del>Gtk2_PHPConfig</del> (kguest)
    * <del>Net_Wifi</del> (cweiske)
    * <del>File_XSPF</del> (doconnor)
    * <del>Net_Whois</del> (doconnor)

  * Followup on [[http://pear.php.net/bugs/search.php?cmd=display&search_for=PHPUnit&php_os=&boolean=0&bug_type=&bug_age=0&bug_updated=0&order_by=ts1&direction=DESC&packagever=&phpver=&limit=30&handle=&assign=&maintain=&showmenu=1&showmenu=1&status=Open|PHPUnit patches]]
  * Make tests pass: 
    * <del>Text_Wiki</del> (justinpatrin)
    * <del>Services_Amazon</del> (Tatsuya Tsuruoka)
    * <del>File_Find</del> (doconnor)
  * <del>Triage the [[http://pear.php.net/bugs/search.php?search_for=&boolean=0&limit=70&order_by=id&direction=DESC&cmd=display&status=Open&bug_type=All&php_os=&packagever=&phpver=&assign=&maintain=&author_email=&bug_age=0&bug_updated=0|latest bugs]]</del> (doconnor: done-ish) 
    * <del>HTTP_Client 1.2.1</del> released (avb)
    * <del>File_XSPF missing package.xml</del> (kguest)
    * <del>MDB2_Schema test coverage</del> (Igor Feghali)

==== Other items needing volunteers ====
  * writing tests for MDB2_Schema bugs (particularly those with patches);
  * <del>I18N_UnicodeString needs to be put in CVS (http://news.php.net/php.pear.dev/50419) (in CVS and fixed by kguest)</del>
  * <del>PHP_Fork into CVS</del> (cweiske)
  * Packages that have the most open bug reports  (http://pear.php.net/bugs/stats.php)
  * <del>Apply Net_Ping patch for bug #14492</del> (patch by [[shupp]], applied by  kguest)
  * Docs for Validate packages - ([[amir]] ?)
  * Payment_DTA into CVS (doconnor consulting with lead)
  * <del>Fix Validate_AU unit tests</del> (doconnor was shirking, but Validate_AU-0.1.2 now exists)

==== Notes: The best way to write unit tests for CVS ====
Say you have a package, Foo_Bar_Item

If you structure you package as:

CVS dir: 
<code>/Foo_Bar_Item/</code>

Structure:
<code>/Foo_Bar_Item/Foo/Bar/Item.php
/Foo_Bar_Item/tests/AllTests.php</code>

Then it makes doing 
<code>
you@locahost:~/pear$ cvs update
you@locahost:~/pear$ cd Foo_Bar_Item
you@locahost:~/pear/Foo_Bar_Item$ phpunit tests/
</code>
... work just fine with the existing require_once patterns; and when you install the package, the tests should still run.

Otherwise, if you have packages with different structure, you will often have to:

<code>
you@locahost:~/pear$ cvs update
you@locahost:~/pear$ cd Foo_Bar_Item
you@locahost:~/pear/Foo_Bar_Item$ pear package
you@locahost:~/pear/Foo_Bar_Item$ sudo pear install -fa Foo_Bar_Item-0.1.0.tgz
you@locahost:~/pear/Foo_Bar_Item$ phpunit tests/
</code>

==== Unit test results ====
These unit test results are generated overnight / on demand on doconnor's development copy; so may be somewhat overly passing at the moment :)
  * [[http://docs.google.com/Doc?id=acm3gmkf7hk_17dfbp7bsw|Unit test results - Oct 23rd]] 518 / 3195 tests fail or error
  * Tests: 3197, Assertions: 20496, Failures: 398, Errors: 48, Incomplete: 86.
  * Tests: 3198, Assertions: 20496, Failures: 274, Errors: 48, Incomplete: 86.