Only for default lang tests. Everything outside of the specs (it functions)
is evaluated during setup (defining of describe scopes and containing
specs). Only when the tests are being executed, moment.js honors the
settings I've passed along within the context of a given test. That is why
month inside the test always returns the English month descriptor because
it is evaluated during setup.

I could place a ```moment.lang(language);``` within the ```describe```
scope but outside the hooks and specs, before calling
```moment.months()```. That would look prettier indeed.
On May 29, 2014 11:05 PM, "Adam Shaw" <notifications@github.com> wrote:

> In tests/automated/monthNames.js:
>
> > +  });
> > +
> > +  [ 'month', 'agendaDay', 'basicDay' ].forEach(function(viewClass, index, viewClasses) {
> > +    describe('when view is ' + viewClass, function() {
> > +      beforeEach(function() {
> > +        settings.defaultView = viewClass;
> > +      });
> > +
> > +      describe('when lang is default', function() {
> > +        beforeEach(function() {
> > +          settings.lang = 'en';
> > +          moment.lang('en');
> > +        });
> > +
> > +        moment.months().forEach(function(month, index, months) {
> > +          it('should be ' + months[index], function(done) {
>
> picky note: in places where you are doing months[index] and
> moment.months()[index] (below), can't you just be using month?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arshaw/fullcalendar/pull/166/files#r13204584>.
>