You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bugs@httpd.apache.org by bu...@apache.org on 2015/07/22 18:05:50 UTC

[Bug 58172] New: Create tests for mod_session

https://bz.apache.org/bugzilla/show_bug.cgi?id=58172

            Bug ID: 58172
           Summary: Create tests for mod_session
           Product: Apache httpd-2
           Version: 2.5-HEAD
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: mod_session
          Assignee: bugs@httpd.apache.org
          Reporter: paul.spangler@ni.com

Currently there aren't any tests for mod_session in httpd-test. I believe
having tests would greatly improve the ability to make fixes to mod_session in
the future by providing a way to verify functionality and detect regressions.

I'm including a patch for 21 tests that covers the directives, hooks, and
optional functions that mod_session provides. It uses a test c-module that
receives session data from the query string and saves session data to a
response header. That way it doesn't have any dependencies on the session
sub-modules.

I've verified the tests pass in trunk and 2.4.x. In 2.2.x (where there is no
mod_session) the tests are skipped. A subset of the tests are skipped if
mod_include is not loaded.

There are two tests that fail with TODO because of PR 56052 and 58171.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


[Bug 58172] Create tests for mod_session

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58172

--- Comment #1 from Paul Spangler <pa...@ni.com> ---
Created attachment 32928
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=32928&action=edit
Proposed tests for mod_session

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


[Bug 58172] Create tests for mod_session

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58172

Edward Lu <Ch...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #2 from Edward Lu <Ch...@gmail.com> ---
Reviewed, looks good. Committed in r1693140.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


[Bug 58172] Create tests for mod_session

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58172

Paul Spangler <pa...@ni.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |PatchAvailable

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org