Feature: unittestAppender about log4js-node HOT 3 CLOSED

Sounds like it would be useful, but the simplest implementation would be just something like this:

var logs = [];
function logger(logEvent) {
  logs.push(logEvent);
}
log4js.addAppender(logger, 'some-category');

As far as contributing, in your fork of log4js-node create a branch for the new work (see http://gitref.org/branching/). Write your new appender, along with some tests, commit to that branch. Then submit a pull request to me (you can do this via github) to merge into the main log4js.

To get things working in Windows, I had to install the following:
http://www.microsoft.com/visualstudio/en-us/products/2010-editions/visual-cpp-express
http://www.python.org/download/releases/2.7.3/

I've also renamed the test files so that "npm test" will now pick them up. You'll need to update your fork with the latest from mine. I've tried it out on Windows 7, and npm test works now.

from log4js-node.

Im doing my first steps on GIThub, so the goals are not supreme
arhitecture, but the flow -
so - Yes, I know the implementation is simple :) that's more or less what I
had in mind.
Now I'm "Injecting" a mock logger that does the same, and it's kind'a
tiering to have it exposed on every utility and inject it in every
suite-setup - so if we could provide something loadable programmatically to
unit-test frameworks, and then queriable for these tests through simple
usable API - we're winners :)

On Tue, Jul 31, 2012 at 7:58 AM, Gareth Jones <
[email protected]

wrote:

Sounds like it would be useful, but the simplest implementation would be
just something like this:

var logs = [];
function logger(logEvent) {
  logs.push(logEvent);
}
log4js.addAppender(logger, 'some-category');

As far as contributing, in your fork of log4js-node create a branch for
the new work (see http://gitref.org/branching/). Write your new appender,
along with some tests, commit to that branch. Then submit a pull request to
me (you can do this via github) to merge into the main log4js.

To get things working in Windows, I had to install the following:

http://www.microsoft.com/visualstudio/en-us/products/2010-editions/visual-cpp-express
http://www.python.org/download/releases/2.7.3/

I've also renamed the test files so that "npm test" will now pick them up.
You'll need to update your fork with the latest from mine. I've tried it
out on Windows 7, and npm test works now.

---

Reply to this email directly or view it on GitHub:
#84 (comment)

from log4js-node.

Cleaning up old issues.

from log4js-node.