fix for bug in FileAppenderFactory where even if archive is set to true it returns a regular file appender instead of rolling file appender

I tried to add a unit test but the best I could come up with was to peek at the internals of a private method. Please suggest a better unit test and I can resubmit.