[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] [PATCH] org-agenda: Add 'none setting for org-agenda-overriding-

From: Nicolas Goaziou
Subject: Re: [O] [PATCH] org-agenda: Add 'none setting for org-agenda-overriding-header
Date: Sun, 10 Sep 2017 14:32:41 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)


Adam Porter <address@hidden> writes:

> I looked at that file and I see that it's pretty basic.

There's a first step in everything.

> I recently put together a sort of testing framework for
> org-super-agenda that makes it pretty easy to develop and run
> consistent tests on agendas:
> https://github.com/alphapapa/org-super-agenda/blob/master/test/test.el
> Would you be interested in a patch that adds something like that to Org?
> It would make it easy to test specific agenda features, and I guess it
> might help with your agenda rewrite.  :)

Thank you.

I didn't try it, but AFAIU, it requires to learn some DSL to write
tests. There are two possible issues: 
- the DSL does not cover all use cases, and therefore gets in the way;
- the DSL implies overhead in writing tests, which is counterproductive.

I didn't test your library, so I cannot tell if your system suffers from
one of the items above. I'm merely pointing out my concerns.

OTOH, I think we need proper tooling to write tests for Org agenda.
Since you put some thought in it, do you think it is feasible to have
something like this, albeit more lightweight (e.g., macros around
regular Org code instead of a DSL)? If so, would you to give it a try?


Nicolas Goaziou

reply via email to

[Prev in Thread] Current Thread [Next in Thread]