[ruby-core:68796] [Ruby trunk - Bug #11045] ruby/test_m17n_comb.rb test_str_crypt tests platform-specific crypt behavior

From: nobu@...
Date: 2015-04-08 01:46:45 UTC
List: ruby-core #68796
Issue #11045 has been updated by Nobuyoshi Nakada.


Charles Nutter wrote:
> 3. Do we really want to be testing linux-specific crypt behavior?

I don't think we want, it's just an implementation regression test.

> I propose that we just make the test skip non-alpha salt unconditionally.

Or we need a standard way to isolate this kind of tests.

----------------------------------------
Bug #11045: ruby/test_m17n_comb.rb test_str_crypt tests platform-specific crypt behavior
https://bugs.ruby-lang.org/issues/11045#change-52066

* Author: Charles Nutter
* Status: Open
* Priority: Normal
* Assignee: 
* ruby -v: trunk
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN
----------------------------------------
I think we should just remove the glibc version guards in test_str_crypt in test/ruby/test_m17n_comb.rb.

1. JRuby can't determine glibc version without making a glibc-specific call (because we have no build-time configure script or similar)
2. On platforms that use glibc but don't report version, this guard won't work.
3. Do we really want to be testing linux-specific crypt behavior?

I propose that we just make the test skip non-alpha salt unconditionally.



-- 
https://bugs.ruby-lang.org/

In This Thread

Prev Next