Source-Changes-D archive

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

Re: CVS commit: src/tests/util/sh



On 11/14/11 3:23 PM, Christos Zoulas wrote:
Module Name:    src
Committed By:   christos
Date:           Mon Nov 14 20:23:29 UTC 2011

Modified Files:
        src/tests/util/sh: Makefile
Added Files:
        src/tests/util/sh: t_evaltested.sh

Log Message:
Add a test for PR/45613 (eval failing in a tested context)

This looks like a poor name for the program: in general, if the test program contains a single test case and both have the same name, that's a bad symptom. Following prior art in this same directory, the program should have probably been named "t_eval" and the single test that it currently has "tested_context". This would leave room for the future addition of eval-related tests, whereas the current naming does not.

Home | Main Index | Thread Index | Old Index